Re: current version: Patch - Have psql show current values for a sequence]

From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
To: Dhanaraj M <Dhanaraj(dot)M(at)Sun(dot)COM>
Cc: pgsql-patches(at)postgresql(dot)org
Subject: Re: current version: Patch - Have psql show current values for a sequence]
Date: 2006-05-04 05:34:11
Message-ID: 19100.1146720851@sss.pgh.pa.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-patches

Dhanaraj M <Dhanaraj(dot)M(at)Sun(dot)COM> writes:
> sorry for sending the old version in the previous mail . Here I attach
> the recent version of the patch file.

Surely this problem does not require adding any server-side code.
Something like "select last_value from <seq>" would be more appropriate;
and it'd have some hope of working with back-version servers.

Also, please use something more helpful than "***" as the column
header. Your urge to add a footnote to explain it shows that you
didn't try hard enough to devise a good header to begin with.

[ btw, both fmgroids.h and fmgrtab.c are generated files. Patching
them is unnecessary and inappropriate. ]

regards, tom lane

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bruno Wolff III 2006-05-04 06:05:23 Re: Warts with SELECT DISTINCT
Previous Message Greg Stark 2006-05-04 05:32:45 Re: Warts with SELECT DISTINCT

Browse pgsql-patches by date

  From Date Subject
Next Message Heikki Linnakangas 2006-05-04 06:24:15 Re: Page at a time index scan
Previous Message Dhanaraj M 2006-05-04 05:19:27 current version: Patch - Have psql show current values for a sequence]