Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist - Mailing list pgsql-hackers

From Andres Freund
Subject Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist
Date
Msg-id 20130125132240.GE4289@awork2.anarazel.de
Whole thread Raw
In response to Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist  (Bruce Momjian <bruce@momjian.us>)
Responses Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist
List pgsql-hackers
On 2013-01-24 20:45:20 -0500, Bruce Momjian wrote:
> On Sun, Sep  2, 2012 at 05:40:54PM +0000, james@illusorystudios.com wrote:
> > The following bug has been logged on the website:
> > 
> > Bug reference:      7515
> > Logged by:          James Bellinger
> > Email address:      james@illusorystudios.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



pgsql-hackers by date:

Previous
From: Pavel Stehule
Date:
Subject: Re: Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist
Next
From: Bruce Momjian
Date:
Subject: Re: [BUGS] BUG #7515: DROP TABLE IF EXISTS fails if schema does not exist