| From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
|---|---|
| To: | Bruce Momjian <bruce(at)momjian(dot)us> |
| Cc: | Robert Haas <robertmhaas(at)gmail(dot)com>, Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, Ashutosh Bapat <ashutosh(dot)bapat(at)enterprisedb(dot)com>, Fabrízio Mello <fabriziomello(at)gmail(dot)com>, Pgsql Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: Add support to COMMENT ON CURRENT DATABASE |
| Date: | 2017-01-09 19:52:46 |
| Message-ID: | 20170109195246.v5qc6ksvgdddn557@alvherre.pgsql |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
Bruce Momjian wrote:
> On Mon, Jan 9, 2017 at 01:34:03PM -0500, Robert Haas wrote:
> > On Fri, Jan 6, 2017 at 7:29 PM, Peter Eisentraut
> > <peter(dot)eisentraut(at)2ndquadrant(dot)com> wrote:
> > > On 1/3/17 11:52 PM, Ashutosh Bapat wrote:
> > >> We will need to make CURRENT_DATABASE a reserved keyword. But I like
> > >> this idea more than COMMENT ON CURRENT DATABASE.
> > >
> > > We already have the reserved key word CURRENT_CATALOG, which is the
> > > standard spelling. But I wouldn't be bothered if we made
> > > CURRENT_DATABASE somewhat reserved as well.
> >
> > Maybe I'm just lacking in imagination, but what's the argument against
> > spelling it CURRENT DATABASE? AFAICS, that doesn't require reserving
> > anything new at all, and it also looks more SQL-ish to me. SQL
> > generally tries to emulate English, and I don't normally
> > speak_hyphenated_words.
>
> I assume it is to match our use of CURRENT_USER as having special
> meaning.
CURRENT_USER is a standards-mandated keyword, but CURRENT_DATABASE is
not. The closest thing SQL has is CURRENT_CATALOG, which is the string
that identifies the "current default catalog". This would lead us to
COMMENT ON DATABASE CURRENT_CATALOG
Do we want that spelling? It looks ugly to me.
--
Álvaro Herrera https://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Remote DBA, Training & Services
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Peter Eisentraut | 2017-01-09 20:03:56 | Re: sequence data type |
| Previous Message | Peter Eisentraut | 2017-01-09 19:50:23 | Re: Proposal for changes to recovery.conf API |