From: | Justin Pryzby <pryzby(at)telsasoft(dot)com> |
---|---|
To: | Andres Freund <andres(at)anarazel(dot)de> |
Cc: | Michael Paquier <michael(at)paquier(dot)xyz>, Kyotaro Horiguchi <horikyota(dot)ntt(at)gmail(dot)com>, nathandbossart(at)gmail(dot)com, pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: fix stats_fetch_consistency value in postgresql.conf.sample |
Date: | 2023-03-30 04:03:59 |
Message-ID: | ZCUKL22GutwGrrZk@telsasoft.com |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On Wed, Jul 13, 2022 at 04:49:00PM -0700, Andres Freund wrote:
> On 2022-07-14 08:46:02 +0900, Michael Paquier wrote:
> > On Wed, Jul 13, 2022 at 12:30:00PM -0500, Justin Pryzby wrote:
> > > How did you make this list ? Was it by excluding things that failed for you ?
> > >
> > > cfbot is currently failing due to io_concurrency on windows.
> > > I think there are more GUC which should be included here.
> > >
> > > http://cfbot.cputube.org/kyotaro-horiguchi.html
> >
> > FWIW, I am not really a fan of making this test depend on a hardcoded
> > list of GUCs.
>
> I wonder if we should add flags indicating platform dependency etc to guc.c?
> That should allow to remove most of them?
Michael commented on this, but on another thread, so I'm copying and
pasting it here.
On Thu, Mar 23, 2023 at 08:59:57PM -0500, Justin Pryzby wrote:
> On Fri, Mar 24, 2023 at 10:24:43AM +0900, Michael Paquier wrote:
> > >> * Check consistency of GUC defaults between .sample.conf and pg_settings.boot_val
> > > - It looks like this was pretty active until last October and might
> > > have been ready to apply at least partially? But no further work or
> > > review has happened since.
> >
> > FWIW, I don't find much appealing the addition of two GUC flags for
> > only the sole purpose of that,
>
> The flags seem independently interesting - adding them here follows
> a suggestion Andres made in response to your complaint.
> 20220713234900(dot)z4rniuaerkq34s4v(at)awork3(dot)anarazel(dot)de
>
> > particularly as we get a stronger
> > dependency between GUCs that can be switched dynamically at
> > initialization and at compile-time.
>
> What do you mean by "stronger dependency between GUCs" ?
I'm still not clear what that means ?
I updated the patch to handle the GUC added at 1671f990d.
--
Justin
Attachment | Content-Type | Size |
---|---|---|
0001-pg_settings_get_flags-add-DEFAULT_COMPILE-and-DEFAUL.patch | text/x-diff | 12.7 KB |
0002-test-GUC-default-values-in-postgresql.conf.sample.patch | text/x-diff | 2.2 KB |
From | Date | Subject | |
---|---|---|---|
Next Message | Noah Misch | 2023-03-30 04:32:05 | Re: Doc: Rework contrib appendix -- informative titles, tweaked sentences |
Previous Message | Peter Smith | 2023-03-30 03:51:36 | Re: PGdoc: add ID attribute to create_publication.sgml |