From: | Andres Freund <andres(at)2ndquadrant(dot)com> |
---|---|
To: | Bruce Momjian <bruce(at)momjian(dot)us> |
Cc: | james(at)illusorystudios(dot)com, PostgreSQL-development <pgsql-hackers(at)postgreSQL(dot)org> |
Subject: | Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist |
Date: | 2013-01-25 13:22:40 |
Message-ID: | 20130125132240.GE4289@awork2.anarazel.de |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs pgsql-hackers |
On 2013-01-24 20:45:20 -0500, Bruce Momjian wrote:
> On Sun, Sep 2, 2012 at 05:40:54PM +0000, james(at)illusorystudios(dot)com wrote:
> > The following bug has been logged on the website:
> >
> > Bug reference: 7515
> > Logged by: James Bellinger
> > Email address: james(at)illusorystudios(dot)com
> > PostgreSQL version: 9.1.5
> > Operating system: Ubuntu Linux 12.04 Server
> > Description:
> >
> > If the table being referenced has a schema in its name, and the schema does
> > not exist, DROP TABLE IF EXISTS will have an ERROR instead of a NOTICE.
> >
> > So for instance,
> > DROP TABLE IF EXISTS bar;
> > This is a NOTICE if bar does not exist.
> >
> > DROP TABLE IF EXISTS foo.bar;
> > This is an ERROR if foo does not exist, even though that implies bar does
> > not exist which means it should be a NOTICE.
> >
> > Saw this because it was making a drop/recreate transaction fail on me, after
> > I changed some code to use a schema.
>
> I looked at this bug report from September. The problem is that
> LookupExplicitNamespace() doesn't have a missing_ok parameter, even
> though get_namespace_oid(), which it calls, does. By adding a
> missing_ok parameter and passing it cleanly, I fixed the problem:
>
> test=> DROP TABLE IF EXISTS foo.bar;
> NOTICE: table "bar" does not exist, skipping
> DROP TABLE
>
> Patch attached.
That makes it a HEAD only fix though. I personally don't have a problem
with that, but others might...
Greetings,
Andres Freund
--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Bruce Momjian | 2013-01-25 13:26:51 | Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist |
Previous Message | Andres Freund | 2013-01-25 13:20:25 | Re: cache lookup failed from empty plpythonu function |
From | Date | Subject | |
---|---|---|---|
Next Message | Simon Riggs | 2013-01-25 13:26:30 | Re: Skip checkpoint on promoting from streaming replication |
Previous Message | Pavel Stehule | 2013-01-25 13:11:43 | Re: Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist |