<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jul 7, 2021 at 1:54 PM 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">> On 2021/07/05 11:27, Tatsuo Ishii wrote:<br>
>> Currently sample files of pgpool.conf under src/sample have all the<br>
>> configuration parameters regardless the clustering mode. For example,<br>
>> "replicate_select" is not applied to the streaming replication mode<br>
>> but it's still in the config file and maybe a noise for the streaming<br>
>> replication mode users. Attached is a PoC patch to remove those<br>
>> unnecessary parameters in the config files. For now only config files<br>
>> for the streaming replication mode and the snapshot isolation mode are<br>
>> included in the patch.<br>
> <br>
> It seems good to remove noisy parameters in the config file.<br>
> <br>
> I think it's good to comment out unnecessary parameters instead of<br>
> removing it completely. Like postgres. I feel that pgpool has more<br>
> declared parameters than postgres in the config file.<br>
<br>
One thing I am concerned is, PostgreSQL's comment represents that<br>
particular parameter's default value, not that the parameter is not<br>
necessary. But unlike Pgpool-II, PostgreSQL does not provide<br>
purpose-specific configuration files. Thus commenting out parameters<br>
could be a good idea for Pgpool-II.<br></blockquote><div><br></div><div>I think removing the no-op (not applicable) parameters altogather for specifi</div><div>clustring mode from the respective sampe config files is the right choice.</div><div><br></div><div>On top of that I think we should also comment out all configuration</div><div>parameters ( similar to PostgreSQL) so that the default values for each</div><div>parameter gets picked up automatically. And if user wants to change some</div><div>specific parameter value they can uncomment that particular line and change</div><div>the value as per the requirement.</div><div>IMHO Doing these two things will make the configure files less cluttered and</div><div>easier to read.</div><div><br></div><div>Also, If we comment out all config parameter in sample files, Then in future we can</div><div>enhance the configuration module to store the infomation about each parameter</div><div>that which clustering modes this variable is applicable for, and if some config</div><div>file sets the value of a parameter that is not applicalble for current mode we</div><div>can log the WARNING message which could be helpful to reolve confusions</div><div>in some user scenarios.</div><div><br></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>
Best regards,<br>
--<br>
Tatsuo Ishii<br>
SRA OSS, Inc. Japan<br>
English: <a href="http://www.sraoss.co.jp/index_en.php" rel="noreferrer" target="_blank">http://www.sraoss.co.jp/index_en.php</a><br>
Japanese:<a href="http://www.sraoss.co.jp" rel="noreferrer" target="_blank">http://www.sraoss.co.jp</a><br>
_______________________________________________<br>
pgpool-hackers mailing list<br>
<a href="mailto:pgpool-hackers@pgpool.net" target="_blank">pgpool-hackers@pgpool.net</a><br>
<a href="http://www.pgpool.net/mailman/listinfo/pgpool-hackers" rel="noreferrer" target="_blank">http://www.pgpool.net/mailman/listinfo/pgpool-hackers</a><br>
</blockquote></div></div>