Re: [HACKERS] A GUC to prevent leader processes from running subplans? - Mailing list pgsql-hackers

From Robert Haas
Subject Re: [HACKERS] A GUC to prevent leader processes from running subplans?
Date
Msg-id CA+TgmoZFtGUj5k=ak04fmBy1b=1M7h7v0Kt13m_MborcguBAxg@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] A GUC to prevent leader processes from running subplans?  (Thomas Munro <thomas.munro@enterprisedb.com>)
Responses Re: [HACKERS] A GUC to prevent leader processes from running subplans?  (Thomas Munro <thomas.munro@enterprisedb.com>)
List pgsql-hackers
On Sat, Nov 11, 2017 at 10:48 PM, Thomas Munro
<thomas.munro@enterprisedb.com> wrote:
> How about parallel_leader_participation = on|off?  The attached
> version has it that way, and adds regression tests to exercise on, off
> and off-but-couldn't-start-any-workers for both kinds of gather node.

This looks mostly fine to me, but I think the documentation is strange:

+        Allows the leader process to execute the query plan under
+        <literal>Gather</literal> and <literal>Gather Merge</literal> nodes
+        instead of waiting for worker processes.  The default is
+        <literal>on</literal>.  Setting this value to <literal>on</literal>
+        can cause the leader process to begin producing tuples sooner instead
+        of waiting for worker processes to start up, but might in some cases
+        also cause workers to become blocked waiting for the leader to clear
+        tuple queues.

This documentation would seem exactly right to me if the default value
were off, but as it is it seems kinda backwards, because it's
explaining why you might want to set the value to what is anyway the
default.  Also, it's always possible for the workers to become blocked
waiting for the leader, regardless of how this GUC is set.  It becomes
more likely when this turned off, but that's not quite the same thing.

-- 
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company


pgsql-hackers by date:

Previous
From: Andres Freund
Date:
Subject: Re: [HACKERS] pg_upgrade to clusters with a different WAL segmentsize
Next
From: Robert Haas
Date:
Subject: Re: [HACKERS] GatherMerge misses to push target list