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.
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 |
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 |