Re: Patch for BUG #2073: Can't drop sequence when created

From: Bruce Momjian <pgman(at)candle(dot)pha(dot)pa(dot)us>
To: Dhanaraj M <Dhanaraj(dot)M(at)Sun(dot)COM>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: Patch for BUG #2073: Can't drop sequence when created
Date: 2006-04-29 16:43:54
Message-ID: 200604291643.k3TGhsj22229@candle.pha.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-patches


Patch applied. Thanks.

---------------------------------------------------------------------------

Dhanaraj M wrote:
> Hi
>
> I send the appropriate patch for bug #2073. This fix disallows to change the default sequence.
> I ran the regression test and passed. The bug details are given below.
> I am awaiting to answer for any further clarifications.
>
> ===================================================================
> > Bug reference: 2073
> > Logged by: Aaron Dummer
> > Email address: aaron ( at ) dummer ( dot ) info
> > PostgreSQL version: 8.0.3
> > Operating system: Debian Linux
> > Description: Can't drop sequence when created via SERIAL column
> > Details:
> >
> > If I create a table named foo with a column named bar, column type SERIAL,
> > it auto-generates a sequence named foo_bar_seq. Now if I manually create a
> > new sequence called custom_seq, and change the default value of foo.bar to
> > reference the new sequence, I still can't delete the old sequence
> > (foo_bar_seq).
> >
> > In other words, from a user's point of view, the foo table is no longer
> > dependent on the foo_bar_seq, yet the system still sees it as dependent.
> >
> > I brought this topic up on the #postgresql IRC channel and the behavior was
> > confirmed by AndrewSN, scampbell_, and mastermind.
>
> Right. We have this TODO item:
>
> * %Disallow changing default expression of a SERIAL column?
>
> which would prevent you from changing the default expression for a
> SERIAL column. So the answer is, don't do that, and in the future, we
> might prevent it.
>
> --
> Bruce Momjian
>
> ==================================================================
>

>
> ---------------------------(end of broadcast)---------------------------
> TIP 5: don't forget to increase your free space map settings

--
Bruce Momjian http://candle.pha.pa.us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Responses

Browse pgsql-patches by date

  From Date Subject
Next Message Tom Lane 2006-04-29 19:07:30 Re: Cleaning up multiply-defined-symbol warnings on OS X
Previous Message Bruce Momjian 2006-04-29 16:35:02 Re: Win32 semaphore patch