From: | Josh Kupershmidt <schmiddy(at)gmail(dot)com> |
---|---|
To: | Robert Haas <robertmhaas(at)gmail(dot)com> |
Cc: | Andreas Karlsson <andreas(at)proxel(dot)se>, 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 04:19:42 |
Message-ID: | AANLkTi=+e0hcGg0dTbwrwNvwOPs1j0iTW8RFqrakb7HN@mail.gmail.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jan 19, 2011 at 9:09 PM, Robert Haas <robertmhaas(at)gmail(dot)com> wrote:
> 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.
OK, changed. I've shuffled the columns slightly so that handlers and
Validator columns are next to each other.
> 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
Fixed, updated patch attached. Though, reading the description of
lanispl on that page, I wonder if "user-defined language" correctly
describes plpgsql these days, which comes installed by default.
Josh
Attachment | Content-Type | Size |
---|---|---|
psql_languages.v9.patch | text/x-patch | 6.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Fujii Masao | 2011-01-20 04:23:38 | Re: pg_basebackup for streaming base backups |
Previous Message | Nathan Boley | 2011-01-20 04:12:59 | Re: estimating # of distinct values |