On Tue, Jan 25, 2022 at 09:44:26PM -0600, Justin Pryzby wrote: > It seems like an arbitrary and short-sighted policy to expose a handful of > flags in the view for the purpose of retiring ./check_guc, but not expose > other > flags, because we thought we knew that no user could ever want them. > > We should either expose all the flags, or should put them into an undocumented > function. Otherwise, how would we document the flags argument ? "Shows some > of the flags" ? An undocumented function avoids this issue.
My vote would be to have a documented function, with a minimal set of the flags exposed and documented, with the option to expand that in the future. COMPUTED and EXPLAIN are useful, and allow some of the automated tests to happen. NOT_IN_SAMPLE and GUC_NO_SHOW_ALL are less useful for the user, and are more developer oriented, but are useful for the tests. So having these four seem like a good first cut. -- Michael
signature.asc
Description: PGP signature
