Re: BUG #7873: pg_restore --clean tries to drop tables that don't exist

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. +

In response to

Responses

Browse pgsql-bugs by date

  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

Browse pgsql-hackers by date

  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)