<div dir="ltr">Hi<div><br></div><div>Thanks Ishii-San and Emond for review and verification.</div><div><br></div><div>I have committed the fix for 4_2 and master branches </div><div><a href="https://git.postgresql.org/gitweb/?p=pgpool2.git;a=commitdiff;h=3e8807f49af665e66da404c632e4a17819e826ec">https://git.postgresql.org/gitweb/?p=pgpool2.git;a=commitdiff;h=3e8807f49af665e66da404c632e4a17819e826ec</a><br></div><div><br></div><div>Regards</div><div>--</div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 10, 2021 at 12:56 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">Hi Usama,<br>
<br>
Yeah, only fairly new gcc can detect the warning.<br>
F.Y.I. mine is gcc 9.3.0 on Ubuntu 20.<br>
<br>
> Hi Ishii San,<br>
> <br>
> Thanks for pointing that out. Unfortunately, GCC version on my MacOS does<br>
> not<br>
> support '-Wmisleading-indentation' so I didn't get any warning.<br>
> <br>
> I will take care of the indentation before committing the patch.<br>
> <br>
> Thanks<br>
> Best regards<br>
> Muhammad Usma<br>
> <br>
> <br>
> On Fri, Sep 10, 2021 at 7:19 AM Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>> wrote:<br>
> <br>
>> Hi Usama,<br>
>><br>
>> Thank you for the patch.<br>
>> You might want to fix below before committing the patch:<br>
>><br>
>> wd_json_data.c: In function ‘parse_wd_exec_cluster_command_json’:<br>
>> wd_json_data.c:934:2: warning: this ‘if’ clause does not guard...<br>
>> [-Wmisleading-indentation]<br>
>>   934 |  if (*args_list)<br>
>>       |  ^~<br>
>> wd_json_data.c:936:3: note: ...this statement, but the latter is<br>
>> misleadingly indented as if it were guarded by the ‘if’<br>
>>   936 |   *args_list = NULL;<br>
>>       |   ^<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>
>> > Hi,<br>
>> ><br>
>> > It was a silly coding mistake that was causing the crash. I have cooked<br>
>> up<br>
>> > a fix and currently testing it.<br>
>> ><br>
>> > Attached are the patches for master and 4.2 branches, If you want to try<br>
>> > these to verify if the fix works for your scenario as well.<br>
>> ><br>
>> > Thanks<br>
>> > Best Regards<br>
>> ><br>
>> ><br>
>> > On Thu, Sep 9, 2021 at 1:15 PM Tatsuo Ishii <<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>> wrote:<br>
>> ><br>
>> >> >> Do you use watchdog and enable follow_primary_command? If so, You are<br>
>> >> >> right. We identified the issue with 4.2.4 and one of Pgpool-II<br>
>> >> >> developers is working on it. Sorry for the inconvenience.<br>
>> >> >><br>
>> >> ><br>
>> >> > Yes, we the watchdog with follow_primary_command, so this might be the<br>
>> >> > issue you identified. Do you already have a patch I might try?<br>
>> >><br>
>> >> Not yet. But Usama, one of Pgpool-II developers, (he is an expert<br>
>> >> on watchdog) is working on it now.<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>
>><br>
</blockquote></div>