Re: describe special values in GUC descriptions more consistently
Daniel Gustafsson <daniel@yesql.se>
From: Daniel Gustafsson <daniel@yesql.se>
To: Nathan Bossart <nathandbossart@gmail.com>
Cc: "David G. Johnston" <david.g.johnston@gmail.com>,
Peter Smith <smithpb2250@gmail.com>,
PostgreSQL Hackers <pgsql-hackers@postgresql.org>
Date: 2025-02-11T22:41:51Z
Lists: pgsql-hackers
> On 11 Feb 2025, at 19:11, Nathan Bossart <nathandbossart@gmail.com> wrote: > I thought about this one a bit, and I actually came to the opposite > conclusion. IMHO it's reasonably obvious that an empty string means that > the file isn't loaded, so there's not much point in stating it in the GUC > description. Instead, I think we should follow the > archive_command/archive_library example and use this space _only_ as a > cross-reference to each other. There's certainly some nuances missed with > this strategy, but that's not unique to this GUC. I don't necessarily disagree with this, but the proposed wording makes it sound sort of like users have to select one or the other. Could it be softened a little like perhaps "An empty string disables, \"ssl_crl_foo\" is still used"? -- Daniel Gustafsson
Commits
-
Adjust auto_explain's GUC descriptions.
- e636da920028 18.0 landed
-
Describe special values in GUC descriptions more consistently.
- 977d865c36cc 18.0 landed