| From: | Robert Haas <robertmhaas(at)gmail(dot)com> |
|---|---|
| To: | Andreas Karlsson <andreas(at)proxel(dot)se> |
| Cc: | Josh Kupershmidt <schmiddy(at)gmail(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org>, Fernando Ike <fike(at)midstorm(dot)org> |
| Subject: | Re: psql: Add \dL to show languages |
| Date: | 2011-01-20 02:09:35 |
| Message-ID: | AANLkTikMnBezrjDmMR2GnAQu7mJUEs-gPkhJc4f+mME-@mail.gmail.com |
| Views: | Raw Message | Whole Thread | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
On Wed, Jan 19, 2011 at 5:47 PM, Andreas Karlsson <andreas(at)proxel(dot)se> wrote:
> The patch looks alright now so I will mark it as ready for committer
> now.
This patch doesn't seem terribly consistent to me - we show the name
of the call handler and the name of the validator, but for the inline
handler we just indicate whether there is one or not. That seems like
something that we should make consistent, and my vote is to show the
name in all cases.
Also, I'm wondering whether the System Language column be renamed to
Internal Language, for consistency with the terminology used here:
http://www.postgresql.org/docs/current/static/catalog-pg-language.html
--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Robert Haas | 2011-01-20 02:12:29 | Re: Use of O_DIRECT only for open_* sync options |
| Previous Message | Fujii Masao | 2011-01-20 02:06:53 | Re: pg_basebackup for streaming base backups |