Re: [PATCH] force_parallel_mode and GUC categories

From: Justin Pryzby <pryzby(at)telsasoft(dot)com>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Bruce Momjian <bruce(at)momjian(dot)us>, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers(at)lists(dot)postgresql(dot)org
Subject: Re: [PATCH] force_parallel_mode and GUC categories
Date: 2021-04-09 03:17:18
Message-ID: 20210409031718.GO6592@telsasoft.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers pgsql-performance

On Fri, Apr 09, 2021 at 10:50:53AM +0900, Michael Paquier wrote:
> On Sat, Apr 03, 2021 at 08:25:46PM -0500, Justin Pryzby wrote:
> > Forking this thread
> > https://www.postgresql.org/message-id/20210403154336.GG29125%40momjian.us
>
> Didn't see this one, thanks for forking.
>
> - {"force_parallel_mode", PGC_USERSET, QUERY_TUNING_OTHER,
> + {"force_parallel_mode", PGC_USERSET, DEVELOPER_OPTIONS,
> And not this one either, as it is mainly a planner thing, like the
> other parameters in the same area.

This is the main motive behind the patch.

Developer options aren't shown in postgresql.conf.sample, which it seems like
sometimes people read through quickly, setting a whole bunch of options that
sound good, sometimes including this one. And in the best case they then ask
on -performance why their queries are slow and we tell them to turn it back off
to fix their issues. This changes to no longer put it in .sample, and calling
it a "dev" option seems to be the classification and mechanism by which to do
that.

--
Justin

ps, Maybe you saw that I'd already resent without including the accidental junk
hunks.

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Bharath Rupireddy 2021-04-09 03:20:29 Re: psql - add SHOW_ALL_RESULTS option
Previous Message Bharath Rupireddy 2021-04-09 03:14:41 Re: Why is Query NOT getting cancelled with SIGINT in PG14?

Browse pgsql-performance by date

  From Date Subject
Next Message Laurenz Albe 2021-04-09 09:15:53 Re: INSERTS waiting with wait_event is "transactionid"
Previous Message Michael Paquier 2021-04-09 01:50:53 Re: [PATCH] force_parallel_mode and GUC categories