Re: FWD: Re: Updated backslash consistency patch

From: Robert Haas <robertmhaas(at)gmail(dot)com>
To: Bruce Momjian <bruce(at)momjian(dot)us>
Cc: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>, Gregory Stark <stark(at)enterprisedb(dot)com>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, Brendan Jurd <direvus(at)gmail(dot)com>, Greg Sabino Mullane <greg(at)turnstep(dot)com>
Subject: Re: FWD: Re: Updated backslash consistency patch
Date: 2009-01-20 04:57:14
Message-ID: 603c8f070901192057m156af3a8oa7a17878095b7fe2@mail.gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> Here is what I hope is a consensus patch. It adds 'A' to show all
> objects, including system ones. It turns out that this is how 'S' works
> now in CVS, but 'S' is unclear because it suggests just system objects;
> 'A' for show 'all' objects seems clearer.

I think it's probably fine for "S" to mean "include system objects"
rather than "show only system objects". Everyone should be relatively
used to "S" by now; I think it's less confusing to keep the same
letter even if the behavior has been adjusted somewhat. Though others
may disagree?

(Although this met with some earlier opposition, mostly from Tom IIRC,
I still think it might be useful to have a setting to control which
types of system objects are displayed by default: the setting could be
interpreted as a list of characters X for which \dX would include
system objects even without S. This would allow people who liked the
old behavior to mostly recreate it.)

...Robert

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Peter Eisentraut 2009-01-20 05:09:34 Re: is 8.4 array_agg() supposed to work with array values?
Previous Message Bruce Momjian 2009-01-20 04:12:30 Re: FWD: Re: Updated backslash consistency patch