Re: Move un-parenthesized syntax docs to "compatibility" for few SQL commands
Melanie Plageman <melanieplageman@gmail.com>
From: Melanie Plageman <melanieplageman@gmail.com>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: Pg Hackers <pgsql-hackers@postgresql.org>, Tom Lane <tgl@sss.pgh.pa.us>
Date: 2023-04-21T23:29:59Z
Lists: pgsql-hackers
Attachments
- v2-0001-Doc-Move-un-parenthesized-syntax-to-compatibility.patch (text/x-patch) patch v2-0001
On Fri, Apr 21, 2023 at 6:55 PM Nathan Bossart <nathandbossart@gmail.com> wrote: > > On Fri, Apr 21, 2023 at 06:29:16PM -0400, Melanie Plageman wrote: > > Over in [2], it was suggested that moving the un-parenthesized syntax to > > the "Compatibility" section of their respective docs pages would be > > okay. Patch attached. > > I think that's reasonable. > > > I left out CLUSTER since that syntax change is so new. > > I think it'd be okay to move this one, too. The parenthesized syntax has > been available since v14, and I suspect CLUSTER isn't terribly widely used, > anyway. (Side note: it looks like "CLUSTER (VERBOSE)" doesn't work, which > seems weird to me.) Attached v2 includes changes for CLUSTER syntax docs. I wasn't quite sure that we can move down CLUSTER [VERBOSE] syntax to the compatibility section since CLUSTER (VERBOSE) doesn't work. This seems like it should work (VACUUM and ANALYZE do). I put extra "[ ]" around the main CLUSTER syntax but I don't know that this is actually the same as documenting that CLUSTER VERBOSE does work but CLUSTER (VERBOSE) doesn't. > Most of these commands have an existing note about the deprecated syntax. > Could those be removed with this change? Ah, yes. Good catch! I have removed these. - Melanie
Commits
-
Doc: move unparenthesized syntaxes for a few commands.
- ab29a7a9c6ff 17.0 landed
-
Support parenthesized syntax for CLUSTER without a table name.
- cdaedfc96d14 17.0 landed