From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Dave Rolsky <autarch(at)urth(dot)org> |
Cc: | pgsql-bugs(at)postgresql(dot)org |
Subject: | Re: BUG #7873: pg_restore --clean tries to drop tables that don't exist |
Date: | 2013-02-15 23:46:37 |
Message-ID: | 20130215234637.GH12030@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
On Fri, Feb 15, 2013 at 04:06:12PM -0600, Dave Rolsky wrote:
> On Fri, 15 Feb 2013, Bruce Momjian wrote:
>
> >On Wed, Feb 13, 2013 at 08:22:43PM +0000, autarch(at)urth(dot)org wrote:
> >>The following bug has been logged on the website:
> >>
> >>Bug reference: 7873
> >>Logged by: Dave Rolsky
> >>Email address: autarch(at)urth(dot)org
> >>PostgreSQL version: 9.2.3
> >>Operating system: Linux
> >>Description:
> >>
> >>When you pass the --clean option to pg_restore it tries to drop tables
> >>without checking if they exist. This results in lots of error output. If
> >>you're running pg_restore via an automated process it's very hard to
> >>distinguish between these "ok" errors and real errors.
> >>
> >>It should be using "DROP TABLE IF EXISTS" and the equivalent for
> >>constraints.
> >
> >Well, I think the question is whether you want error feedback for things
> >that don't exist. I don't really know the answer.
>
> Fair enough. It should probably an option to add "if exists", at
> least. I can't imagine I'm the only using this tool to ship database
> updates around to different machines, some of which may not have new
> tables. I'd really like to be able to know when the restore fails
> versus when it succeeds but is noisy.
All I can say is I don't remember anyone asking for this in the past.
--
Bruce Momjian <bruce(at)momjian(dot)us> http://momjian.us
EnterpriseDB http://enterprisedb.com
+ It's impossible for everything to be true. +
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Farina | 2013-02-15 23:49:10 | Re: BUG #7883: "PANIC: WAL contains references to invalid pages" on replica recovery |
Previous Message | Maciek Sakrejda | 2013-02-15 23:18:37 | Re: BUG #7883: "PANIC: WAL contains references to invalid pages" on replica recovery |
From | Date | Subject | |
---|---|---|---|
Next Message | Kevin Grittner | 2013-02-16 01:01:10 | Re: Materialized views WIP patch |
Previous Message | Daniele Varrazzo | 2013-02-15 23:11:40 | Re: [RFC] ideas for a new Python DBAPI driver (was Re: libpq test suite) |