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

From: Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Robert Haas <robertmhaas(at)gmail(dot)com>, Andres Freund <andres(at)2ndquadrant(dot)com>, Tomas Vondra <tv(at)fuzzy(dot)cz>, PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Re: [BUGS] BUG #7873: pg_restore --clean tries to drop tables that don't exist
Date: 2013-11-28 07:44:23
Message-ID: CAFj8pRDygU89vSbXUic+i9S_qcCufy85TEMeRUbqC6EL+YrJTg@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-bugs pgsql-hackers

Hello

2013/11/27 Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>

> Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> writes:
> > Actually the IF EXISTS in DROP TABLE now applies to the schema as
> > well. Unfortunately there is currently no consistency across the
> > various DROP commands --- some tolerate a non-existent schema, while
> > others error out.
>
> Yeah. I think now that we've had this discussion, we should make them
> all tolerate a non-existent schema. I'm fine with having that happen
> over a series of patches rather than all at once though.
>
> > Also amongst those that tolerate a non-existent
> > schema, the resulting notices are not consistent --- some report the
> > schema-qualified object name, while others just report the local
> > object name.
>
> Less excited about this part, but on the whole I'd vote for the "schema
> "no_such_schema" does not exist" wording in cases where the schema isn't
> there. The other way is less specific for no very good reason.
>

can be used this behave (see attached patch, please)?

if it is correct, I'll work on second patch, that unify check and notices
for other DROP IF EXISTS statements.

Regards

Pavel

>
> regards, tom lane
>

Attachment Content-Type Size
drop-trigger-if-exists.patch text/x-patch 8.3 KB

In response to

Browse pgsql-bugs by date

  From Date Subject
Next Message Andres Freund 2013-11-28 09:53:41 Re: BUG #8632: file "pg_subtrans/CEC0" doesn't exist, reading as zeroes
Previous Message Tomonari Katsumata 2013-11-28 04:00:33 Re: BUG #8434: Why does dead lock occur many times ?

Browse pgsql-hackers by date

  From Date Subject
Next Message Boszormenyi Zoltan 2013-11-28 07:47:29 Re: Modify the DECLARE CURSOR command tag depending on the scrollable flag
Previous Message Alexander Korotkov 2013-11-28 07:19:06 Re: GIN improvements part 1: additional information