Re: Have psql show current sequnce values - (Resubmission)

From: Alvaro Herrera <alvherre(at)commandprompt(dot)com>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: Dhanaraj M <Dhanaraj(dot)M(at)Sun(dot)COM>, pgsql-patches(at)postgresql(dot)org
Subject: Re: Have psql show current sequnce values - (Resubmission)
Date: 2007-02-09 00:24:13
Message-ID: 20070209002413.GT24069@alvh.no-ip.org
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Bruce Momjian wrote:
>
> Your patch has been added to the PostgreSQL unapplied patches list at:
>
> http://momjian.postgresql.org/cgi-bin/pgpatches
>
> It will be applied as soon as one of the PostgreSQL committers reviews
> and approves it.

I think the SELECT query is short on quoting and schema-qualification
for the target sequence.

Also it should probably be adjusted to use strlcpy instead of strcpy.
Coding style seems a bit off. Also, since this is likely to be
expensive, it may be better to show the value only on \dS+, not plain
\dS.

Not sure what else. That's what jumps at me.

I think we discussed replacing the current sequence representation with
a single relation that would contain all sequences in the database, so
that you could do "select * from pg_sequence" and get all the values in
one go. The idea was considered not implementable at the time due to us
not having "non transactional relations", but I'd suggest adding it to
the TODO so that we don't forget later.

--
Alvaro Herrera http://www.CommandPrompt.com/
The PostgreSQL Company - Command Prompt, Inc.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Jan Wieck 2007-02-09 00:28:28 Re: Proposal: Commit timestamp
Previous Message Simon Riggs 2007-02-09 00:07:10 Re: HOT for PostgreSQL 8.3

Browse pgsql-patches by date

  From Date Subject
Next Message Bruce Momjian 2007-02-09 00:32:19 Re: Have psql show current sequnce values - (Resubmission)
Previous Message Tom Lane 2007-02-08 23:45:38 Re: [pgsql-patches] Phantom Command IDs, updated patch