From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Michael Paquier <michael(at)paquier(dot)xyz> |
Cc: | Peter Eisentraut <peter(dot)eisentraut(at)enterprisedb(dot)com>, pgsql-hackers(at)lists(dot)postgresql(dot)org, Bruce Momjian <bruce(at)momjian(dot)us>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> |
Subject: | Re: GUC flags |
Date: | 2021-12-29 02:32:40 |
Message-ID: | 20211229023240.GM24477@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Thu, Dec 09, 2021 at 09:53:23AM -0600, Justin Pryzby wrote:
> On Thu, Dec 09, 2021 at 05:17:54PM +0900, Michael Paquier wrote:
> > On Wed, Dec 08, 2021 at 01:23:51PM +0100, Peter Eisentraut wrote:
> > > I wasn't really aware of this script either. But I think it's a good idea
> > > to have it. But only if it's run automatically as part of a test suite run.
> >
> > Okay. If we do that, I am wondering whether it would be better to
> > rewrite this script in perl then, so as there is no need to worry
> > about the compatibility of grep. And also, it would make sense to
> > return a non-zero exit code if an incompatibility is found for the
> > automation part.
>
> One option is to expose the GUC flags in pg_settings, so this can all be done
> in SQL regression tests.
>
> Maybe the flags should be text strings, so it's a nicer user-facing interface.
> But then the field would be pretty wide, even though we're only adding it for
> regression tests. The only other alternative I can think of is to make a
> sql-callable function like pg_get_guc_flags(text guc).
Rebased on cab5b9ab2c066ba904f13de2681872dcda31e207.
And added 0003, which changes to instead exposes the flags as a function, to
avoid changing pg_settings and exposing internally-defined integer flags in
that somewhat prominent view.
--
Justin
Attachment | Content-Type | Size |
---|---|---|
v3-0001-check_guc-fix-absurd-number-of-false-positives.patch | text/x-diff | 3.9 KB |
v3-0002-Expose-GUC-flags-in-pg_settings-retire-.-check_gu.patch | text/x-diff | 13.9 KB |
v3-0003-f-pg_get_guc_flags.patch | text/x-diff | 5.6 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | tanghy.fnst@fujitsu.com | 2021-12-29 03:16:17 | RE: row filtering for logical replication |
Previous Message | Masahiko Sawada | 2021-12-29 01:34:31 | Re: Allow escape in application_name |