Re: [HACKERS] max_sync_workers_per_subscription is missing in postgresql.conf - Mailing list pgsql-hackers

From Fujii Masao
Subject Re: [HACKERS] max_sync_workers_per_subscription is missing in postgresql.conf
Date
Msg-id CAHGQGwEokE9nJRVgnoWSUvxbUGgT61O2oYJ-C+TaxQoLPTdzFg@mail.gmail.com
Whole thread Raw
In response to Re: [HACKERS] max_sync_workers_per_subscription is missing in postgresql.conf  (Masahiko Sawada <sawada.mshk@gmail.com>)
Responses Re: [HACKERS] max_sync_workers_per_subscription is missing in postgresql.conf  (Masahiko Sawada <sawada.mshk@gmail.com>)
List pgsql-hackers
On Mon, Apr 10, 2017 at 9:39 PM, Masahiko Sawada <sawada.mshk@gmail.com> wrote:
> On Mon, Apr 10, 2017 at 9:32 PM, Petr Jelinek
> <petr.jelinek@2ndquadrant.com> wrote:
>> On 10/04/17 07:16, Masahiko Sawada wrote:
>>> Hi all,
>>>
>>> Attached a patch for $subject.
>>>
>>> I added this parameter into "Asynchronous Behavior" section of
>>> "RESOURCE" section. But GUC parameter for subscriber now is written in
>>> this section, in spite of there is "REPLICATION" section. I think that
>>> we can coordinate these parameters to not confuse user. For example in
>>> documentation, these parameters are described in "19.6.4. Subscribers"
>>> section of "19.6. Replication" section. Thought?

Yes, I think that we should not only add the parameter into
postgresql.conf.sample
but also

- add REPLICATION_SUBSCRIBERS into config_group
- mark max_logical_replication_workers and max_sync_workers_per_subscription
  as REPLICATION_SUBSCRIBERS parameters, in guc.c
- move those parameters into "Subscribers" section in postgresql.conf.sample

The attached patch does these.

Regards,

-- 
Fujii Masao

-- 
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

Attachment

pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: [HACKERS] GCC 7 warnings
Next
From: Tom Lane
Date:
Subject: Re: [HACKERS] src/interfaces/libpq shipping nmake-related Makefiles