From: | Dean Rasheed <dean(dot)a(dot)rasheed(at)gmail(dot)com> |
---|---|
To: | Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> |
Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org>, Peter Eisentraut <peter_e(at)gmx(dot)net> |
Subject: | Re: Re: [BUGS] BUG #7873: pg_restore --clean tries to drop tables that don't exist |
Date: | 2013-12-04 08:56:15 |
Message-ID: | CAEZATCVEZFCTkhr2k96cy5WDnq6B8iTh1tkVTZLdrw17Aedc4Q@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
On 2 December 2013 04:55, Pavel Stehule <pavel(dot)stehule(at)gmail(dot)com> wrote:
> Hello
>
> it looks well, thank you
>
> Regards
>
> Pavel
>
I've been thinking about this some more, and there's another case that
concerns me slightly. We're now making some of the DROP...IF EXISTS
commands tolerate non-existent types as well as non-existent schemas
--- functions, aggregates, casts and operators all have type names in
their specifications. Of course it's possible that the type is missing
because it was in a schema that was dropped, so this change seems to
be in spirit of what was discussed, but it seems like a change that
might catch some people out.
I think that, on balance, it is a sensible change, since if the type
doesn't exist, the dependent object can't exist either, so DROP...IF
EXISTS shouldn't be raising an error. However, I wonder if we should
be issuing a more specific NOTICE in this case too --- i.e., check for
non-existent types in the same way as we check for non-existent parent
objects --- type_does_not_exist_skipping() and
type_list_does_not_exist_skipping().
Regards,
Dean
From | Date | Subject | |
---|---|---|---|
Next Message | Pavel Stehule | 2013-12-04 15:02:41 | Re: Re: [BUGS] BUG #7873: pg_restore --clean tries to drop tables that don't exist |
Previous Message | shaunc | 2013-12-04 07:12:09 | BUG #8655: pg_restore problems with materialized views |
From | Date | Subject | |
---|---|---|---|
Next Message | Andres Freund | 2013-12-04 09:22:33 | Re: Time-Delayed Standbys |
Previous Message | Andres Freund | 2013-12-04 08:54:17 | Re: logical changeset generation v6.7 |