<div dir="ltr"><div>Hi Ishii San,</div><div><br></div><div>Thanks for pointing that out. Unfortunately, GCC version on my MacOS does not</div><div>support '-Wmisleading-indentation' so I didn't get any warning.<br></div><div><br></div><div>I will take care of the indentation before committing the patch.</div><div><br></div><div>Thanks</div><div>Best regards</div><div>Muhammad Usma</div><div><br></div>





<br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Sep 10, 2021 at 7:19 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>
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... [-Wmisleading-indentation]<br>
  934 |  if (*args_list)<br>
      |  ^~<br>
wd_json_data.c:936:3: note: ...this statement, but the latter is 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 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>
</blockquote></div></div>