<div dir="ltr"><div>Hi Ishii San</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Jul 23, 2023 at 8:21 AM Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp">ishii@sraoss.co.jp</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">> Hi Usama,<br>
> <br>
> After this commit regression test starts to fail:<br>
> <br>
> testing 012.watchdog_failover_when_quorum_exists...failed.<br>
> testing 013.watchdog_failover_require_consensus...failed.<br>
> :<br>
> :<br>
> <br>
> Attached are logs for the tests.<br>
<br>
I also wonder this:<br>
<br>
>> Config post-processing stage now throws an error if num_init_children is<br>
>> smaller than max_spare_children.<br>
<br>
>> For the Aggressive strategy, max_spare_children number of processes is spawned;<br>
>> for the other two strategies, min_spare_children number of children gets<br>
>> created at startup.<br>
<br>
Shouldn't min_spare_children and max_spare_chidren be ignored if<br>
process_management_mode = static? From the 4.4 doc:<br></blockquote><div><br></div><div>That is the correct way. I have pushed the commit that skips </div><div>max and min spare children config values verification when static process management mode is used.</div><div>Thanks for pointing this out.</div><div><br></div><div>Best regards</div><div>Muhammad Usama</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
-------------------------------------------------------------------------<br>
min_spare_children (integer)<br>
<br>
    Specify the minimum number of spare (idle) child processes to<br>
    keep. If the idle process count falls below min_spare_children,<br>
    Pgpool-II will spawn new child processes unless it hits the total<br>
    allowed child process ceiling (num_init_children). Default value<br>
    is 5.<br>
    This parameter can be changed by reloading the Pgpool-II configurations.<br>
    This parameter is only applicable for dynamic process management mode. <br>
<br>
max_spare_children (integer)<br>
<br>
    Specify the maximum number of spare (idle) child processes to<br>
    keep. If the idle process count increases from max_spare_children,<br>
    Pgpool-II will kill the excessive child processes. Default value<br>
    is 10.<br>
    This parameter can be changed by reloading the Pgpool-II configurations.<br>
    This parameter is only applicable for dynamic process management mode. <br>
-------------------------------------------------------------------------<br>
<br>
> From: Muhammad Usama <<a href="mailto:m.usama@gmail.com" target="_blank">m.usama@gmail.com</a>><br>
> Subject: [pgpool-committers: 9487] pgpool: Disallowing setting the max_spare_children greater than num_ini<br>
> Date: Sat, 22 Jul 2023 14:26:28 +0000<br>
> Message-ID: <<a href="mailto:E1qNDYx-002KBf-7l@gothos.postgresql.org" target="_blank">E1qNDYx-002KBf-7l@gothos.postgresql.org</a>><br>
> <br>
>> Disallowing setting the max_spare_children greater than num_init_children.<br>
>> <br>
>> Config post-processing stage now throws an error if num_init_children is<br>
>> smaller than max_spare_children.<br>
>> Commit also adjusts the number of child processes spawned at startup based on<br>
>> the configured process management strategy.<br>
>> For the Aggressive strategy, max_spare_children number of processes is spawned;<br>
>> for the other two strategies, min_spare_children number of children gets<br>
>> created at startup.<br>
>> <br>
>> Branch<br>
>> ------<br>
>> master<br>
>> <br>
>> Details<br>
>> -------<br>
>> <a href="https://url.avanan.click/v2/___https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=37dc683f7f15cbf4502b338937e54b9da0a92efd___.YXAzOnBlcmNvbmE6YTpnOjM4MzE2YzY5NThhZGU2NjI5N2UzNTJjMzI5NThlYTU2OjY6YzYyZjo5YzYwNWJkM2E5NTYxZTEyMzY3MDczNjU2ZGRhZmU2NDhhYTdmZGQ1NGMwZmNmZWM5YWI4NWMwNjM5YjkyM2M2OnA6VA" rel="noreferrer" target="_blank">https://url.avanan.click/v2/___https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=37dc683f7f15cbf4502b338937e54b9da0a92efd___.YXAzOnBlcmNvbmE6YTpnOjM4MzE2YzY5NThhZGU2NjI5N2UzNTJjMzI5NThlYTU2OjY6YzYyZjo5YzYwNWJkM2E5NTYxZTEyMzY3MDczNjU2ZGRhZmU2NDhhYTdmZGQ1NGMwZmNmZWM5YWI4NWMwNjM5YjkyM2M2OnA6VA</a><br>
>> <br>
>> Modified Files<br>
>> --------------<br>
>> src/config/pool_config_variables.c |  7 +++++++<br>
>> src/main/pgpool_main.c             | 11 ++++++++---<br>
>> 2 files changed, 15 insertions(+), 3 deletions(-)<br>
>> <br>
_______________________________________________<br>
pgpool-hackers mailing list<br>
<a href="mailto:pgpool-hackers@pgpool.net" target="_blank">pgpool-hackers@pgpool.net</a><br>
<a href="https://url.avanan.click/v2/___http://www.pgpool.net/mailman/listinfo/pgpool-hackers___.YXAzOnBlcmNvbmE6YTpnOjM4MzE2YzY5NThhZGU2NjI5N2UzNTJjMzI5NThlYTU2OjY6NGYzZDo4ODllMWYzZjBhYmZkN2EyNzg4Y2Y4MTM4MWQ5MTExMGI5ZTNiZDJlZjc2MmI0ZjlmM2U2MzYyZjY3NmQ1OWExOnA6VA" rel="noreferrer" target="_blank">https://url.avanan.click/v2/___http://www.pgpool.net/mailman/listinfo/pgpool-hackers___.YXAzOnBlcmNvbmE6YTpnOjM4MzE2YzY5NThhZGU2NjI5N2UzNTJjMzI5NThlYTU2OjY6NGYzZDo4ODllMWYzZjBhYmZkN2EyNzg4Y2Y4MTM4MWQ5MTExMGI5ZTNiZDJlZjc2MmI0ZjlmM2U2MzYyZjY3NmQ1OWExOnA6VA</a><br>
</blockquote></div></div>