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: | Raw Message | Whole Thread | 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 |