Re: [PATCHES] Non-colliding auto generated names - Mailing list pgsql-hackers

From Bruce Momjian
Subject Re: [PATCHES] Non-colliding auto generated names
Date
Msg-id 200303051937.h25Jbai25617@candle.pha.pa.us
Whole thread Raw
In response to Re: [PATCHES] Non-colliding auto generated names  (Rod Taylor <rbt@rbt.ca>)
Responses Re: [PATCHES] Non-colliding auto generated names  (Rod Taylor <rbt@rbt.ca>)
List pgsql-hackers
Interesting idea.  The bigger problem is that apps who use the sequence
name also would have problems running after the restore.  Seems we need
column.nextval() so you can increment the sequence without knowing the
sequence name, just the column name.  Of course, this related to this
TODO item:
* Have sequence dependency track use of DEFAULT sequences,  seqname.nextval

Comments?

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

Rod Taylor wrote:
-- Start of PGP signed section.
> On Wed, 2003-03-05 at 10:36, Bruce Momjian wrote:
> > Ewe.  There would no longer be a guaranteed name for the serial column
> > sequence.  Of course, pg_depend has the information, but how do you get
> > at that when you create the dump file, and be _sure_ you are going to
> > hit the right name, especially if you restore only part of the dump.
> > 
> > Seems this kills the idea of this patch.  With 64-byte names, let's see
> > if we still get complaints about name conflicts.
> 
> We are :)
> 
> CREATE TABLE tab (col SERIAL);
> 
> ALTER TABLE tab RENAME TO tab2;
> 
> CREATE TABLE tab (col SERIAL);
> 
> 
> There is a chance we could do something like:
> 
> ALTER SEQUENCE ON table(col) RESTART WITH <value>
> 
> instead of
> 
> SELECT setval("<sequence>", <value>);
> 
> 
> The ALTER SEQUENCE syntax isn't any worse than setval...  I'd be willing
> to implement the feature in order to get this patch accepted.
> 
> Oh, and welcome back!
> 
> > ---------------------------------------------------------------------------
> > 
> > Christopher Kings-Lynne wrote:
> > > OK,
> > > 
> > > I have discovered a problem with my auto-naming patch.  It's do to with
> > > dumping serial columns with pg_dump, eg:
> > > 
> > > --
> > > -- TOC entry 2 (OID 1004551)
> > > -- Name: users_users; Type: TABLE; Schema: public; Owner: chriskl
> > > --
> > > 
> > > CREATE TABLE users_users (
> > >     userid serial NOT NULL,
> > >     firstname character varying(255) NOT NULL,
> > >     lastname character varying(255) NOT NULL,
> > >     email character varying(255) NOT NULL
> > > );
> > > 
> > > -- DATA DUMPED HERE
> > > 
> > > --
> > > -- TOC entry 4 (OID 1004305)
> > > -- Name: users_users_userid_seq; Type: SEQUENCE SET; Schema: public; Owner:
> > > chriskl
> > > --
> > > 
> > > SELECT pg_catalog.setval ('users_users_userid_seq', 126, true);
> > > 
> > > 
> > > How do we fix this problem??  Perhaps instead of a hard-coded sequence
> > > string, we can sub-SELECT for it...?
> > > 
> > > Chris
> > > 
> > > 
> > > 
> -- 
> Rod Taylor <rbt@rbt.ca>
> 
> PGP Key: http://www.rbt.ca/rbtpub.asc
-- End of PGP section, PGP failed!

--  Bruce Momjian                        |  http://candle.pha.pa.us pgman@candle.pha.pa.us               |  (610)
359-1001+  If your life is a hard drive,     |  13 Roberts Road +  Christ can be your backup.        |  Newtown Square,
Pennsylvania19073
 


pgsql-hackers by date:

Previous
From: Josh Berkus
Date:
Subject: Re: Best setup for RAM drive
Next
From: Rod Taylor
Date:
Subject: Re: [PATCHES] Non-colliding auto generated names