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 |
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 ? |
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 |