From: | Bruce Momjian <bruce(at)momjian(dot)us> |
---|---|
To: | Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Cc: | Imran Zaheer <imran(dot)zhir(at)gmail(dot)com>, Laurenz Albe <laurenz(dot)albe(at)cybertec(dot)at>, Peter Eisentraut <peter(at)eisentraut(dot)org>, masano(at)sraoss(dot)co(dot)jp, pgsql-bugs(at)lists(dot)postgresql(dot)org |
Subject: | Re: BUG #18336: Inconsistency in PostgreSQL 16 Documentation for SHOW Command |
Date: | 2024-08-17 19:36:55 |
Message-ID: | ZsD717W96tCNQCAI@momjian.us |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-bugs |
On Sat, Aug 17, 2024 at 02:12:40PM -0400, Tom Lane wrote:
> Imran Zaheer <imran(dot)zhir(at)gmail(dot)com> writes:
> > The website[1] still shows the redundant LC_COLLATE & LC_CTYPE params.
> > Are we planning to update them?
>
> This patch seems to have fallen through the cracks. I thought
I had kept this thread in my mailbox from February.
> I'd go apply it, but on closer inspection the cross-reference
I was trying to apply Peter Eisentraut's patch when gitmaster stopped
working.
> to SET seems to indicate we have some more work to do.
> That's because set.sgml has its own list of "special" names.
> It'd be fine if SHOW recognized exactly the same list of special
> names, but it seems to recognize only some of them:
>
> regression=# set time zone 'America/New_York';
> SET
> regression=# show time zone;
> TimeZone
> ------------------
> America/New_York
> (1 row)
> regression=# set names 'LATIN1';
> SET
> regression=# show names;
> ERROR: unrecognized configuration parameter "names"
>
> Digging in gram.y, I see that VariableShowStmt actually has these options:
>
> SHOW var_name
> | SHOW TIME ZONE
> | SHOW TRANSACTION ISOLATION LEVEL
> | SHOW SESSION AUTHORIZATION
> | SHOW ALL
>
> which is a subset of the special cases in VariableSetStmt (many
> of which don't seem to be documented anywhere, although I suppose
> most or all are derived from the SQL spec). I wonder if we ought
> to try to make that more systematic. I don't see a really good
> reason why SHOW shouldn't have exactly the same list of special
> target-name syntaxes as SET.
Agredd.
> In short: we might end up applying exactly this patch to show.sgml,
> but we'd have to do some work elsewhere to make the cross-ref to
> set.sgml not still be a lie. Maybe we should go ahead and commit
> it as-is anyway, since it's better than what we have.
I think we should apply what Peter supplied, and then we can do a
follow-up patch to synchronize.
--
Bruce Momjian <bruce(at)momjian(dot)us> https://momjian.us
EDB https://enterprisedb.com
Only you can decide what is important to you.
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Gustafsson | 2024-08-17 21:27:43 | Re: TLS session tickets disabled? |
Previous Message | Tom Lane | 2024-08-17 18:12:40 | Re: BUG #18336: Inconsistency in PostgreSQL 16 Documentation for SHOW Command |