Re: [HACKERS] Quick patch: Display sequence owner

2008-12-09 Thread Robert Treat
On Tuesday 09 December 2008 15:49:17 Alvaro Herrera wrote: > Josh Williams wrote: > > On Tue, 2008-12-09 at 09:32 -0500, Tom Lane wrote: > > > I think the place that such information could most naturally be > > > squeezed into psql's \d commands would be to add another type of footer > > > informat

Re: [HACKERS] Quick patch: Display sequence owner

2008-12-09 Thread Alvaro Herrera
Josh Williams wrote: > On Tue, 2008-12-09 at 09:32 -0500, Tom Lane wrote: > > I think the place that such information could most naturally be squeezed > > into psql's \d commands would be to add another type of footer > > information to \dt, eg > > > > Table "foo.bar" > > ... > >

Re: [HACKERS] Quick patch: Display sequence owner

2008-12-09 Thread Jaime Casanova
On Tue, Dec 9, 2008 at 10:46 AM, Josh Williams <[EMAIL PROTECTED]> wrote: > On Tue, 2008-12-09 at 09:32 -0500, Tom Lane wrote: >> I think the place that such information could most naturally be squeezed >> into psql's \d commands would be to add another type of footer >> information to \dt, eg >> >

Re: [HACKERS] Quick patch: Display sequence owner

2008-12-09 Thread Josh Williams
On Tue, 2008-12-09 at 09:32 -0500, Tom Lane wrote: > I think the place that such information could most naturally be squeezed > into psql's \d commands would be to add another type of footer > information to \dt, eg > > Table "foo.bar" > ... > Indexes: > "ba

Re: [HACKERS] Quick patch: Display sequence owner

2008-12-09 Thread Tom Lane
Alvaro Herrera <[EMAIL PROTECTED]> writes: > Josh Williams wrote: >> The patch adds a query against pg_depend, then fakes an extra column >> "owned_by" in the output: > Please send a context diff (diff -c) Don't bother --- it's a really bad idea as designed anyway. owned_by is not a column of a

Re: [HACKERS] Quick patch: Display sequence owner

2008-12-09 Thread Alvaro Herrera
Josh Williams wrote: > The patch adds a query against pg_depend, then fakes an extra column > "owned_by" in the output: Please send a context diff (diff -c) > Index: src/bin/psql/describe.c > === > RCS file: /projects/cvsroot/pgsql

Re: [HACKERS] Quick patch: Display sequence owner

2008-12-08 Thread David Rowley
Josh Williams wrote: > Hi folks, > > Was recently poked and reminded that this patch may be of interest to > the community. It was mostly done as an academic exercise, just to see > how it works, and so it has a rather hackish feel. The patch adds the > sequence owner, if available, to psql's

[HACKERS] Quick patch: Display sequence owner

2008-12-06 Thread Josh Williams
Hi folks, Was recently poked and reminded that this patch may be of interest to the community. It was mostly done as an academic exercise, just to see how it works, and so it has a rather hackish feel. The patch adds the sequence owner, if available, to psql's \d output, as suggested in a recen