<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Tue, Sep 24, 2019 at 1:22 PM Tatsuo Ishii &lt;<a href="mailto:ishii@sraoss.co.jp">ishii@sraoss.co.jp</a>&gt; 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>
But actually your commit deeply modifies not only watchdog, but other<br>
modules.  I worry about the side effects those modificarions other<br>
than watchdog modules.<br></blockquote><div><br></div><div>The modifications other than in the watchdog module are mostly cosmetic in nature and done to segregate the socket stream from pool stream.</div><div>Having said that, I agree with your point that there could be some unforeseen side effects and its better to keep it for the next version.</div><div><br></div><div>Thanks</div><div>Best Regards</div><div>Muhammad Usama</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
&gt; Hi Ishii-San,<br>
&gt; <br>
&gt; Although we are very late in the release cycle. But I was wondering since<br>
&gt; wd_cli is a separate utility so we can also push it to the V4_1_STABLE<br>
&gt; branch<br>
&gt; so that we can release it along with 4.1 GA, otherwise, it will take quite<br>
&gt; a long before someone is able to use it.<br>
&gt; <br>
&gt; What are your thoughts?<br>
&gt; <br>
&gt; Best Regards<br>
&gt; Muhammad Usama<br>
&gt; <br>
&gt; On Tue, Sep 24, 2019 at 1:08 PM Muhammad Usama &lt;<a href="mailto:m.usama@gmail.com" target="_blank">m.usama@gmail.com</a>&gt; wrote:<br>
&gt; <br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt; On Tue, Sep 24, 2019 at 12:50 PM Tatsuo Ishii &lt;<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>&gt; wrote:<br>
&gt;&gt;<br>
&gt;&gt;&gt; &gt; Hi Tatsuo<br>
&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt; I have not changed/removed the &#39;lifecheck started&quot; message and I am<br>
&gt;&gt;&gt; getting<br>
&gt;&gt;&gt; &gt; clean<br>
&gt;&gt;&gt; &gt; regression including this test case.<br>
&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt; testing 028.watchdog_enable_consensus_with_half_votes...ok.<br>
&gt;&gt;&gt; &gt; out of 1 ok:1 failed:0 timeout:0<br>
&gt;&gt;&gt; &gt;<br>
&gt;&gt;&gt; &gt; Can you send me the pgpool.log for a failed test case? It could be some<br>
&gt;&gt;&gt; &gt; other issue that was<br>
&gt;&gt;&gt; &gt; preventing the pgpool-II to start up in time.<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Strange. This time the test has passed. Will notify you if I get the<br>
&gt;&gt;&gt; failure again (along with pgpool.log).<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Sure, Thanks<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;&gt; BTW, I get compiler warning after your commit:<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; psprintf.c:139:45: warning: implicit declaration of function ‘strerror’;<br>
&gt;&gt;&gt; did you mean ‘perror’? [-Wimplicit-function-declaration]<br>
&gt;&gt;&gt;    fprintf(stderr, &quot;vsnprintf failed: %s\n&quot;, strerror(errno));<br>
&gt;&gt;&gt;                                              ^~~~~~~~<br>
&gt;&gt;&gt;                                              perror<br>
&gt;&gt;&gt; psprintf.c:139:39: warning: format ‘%s’ expects argument of type ‘char<br>
&gt;&gt;&gt; *’, but argument 3 has type ‘int’ [-Wformat=]<br>
&gt;&gt;&gt;    fprintf(stderr, &quot;vsnprintf failed: %s\n&quot;, strerror(errno));<br>
&gt;&gt;&gt;                                       ~^     ~~~~~~~~~~~~~~~<br>
&gt;&gt;&gt;                                       %d<br>
&gt;&gt;&gt;<br>
&gt;&gt;&gt; Sorry about that. I will take care of this one.<br>
&gt;&gt;<br>
&gt;&gt;<br>
&gt;&gt;&gt; Best regards,<br>
&gt;&gt;&gt; --<br>
&gt;&gt;&gt; Tatsuo Ishii<br>
&gt;&gt;&gt; SRA OSS, Inc. Japan<br>
&gt;&gt;&gt; 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>
&gt;&gt;&gt; Japanese:<a href="http://www.sraoss.co.jp" rel="noreferrer" target="_blank">http://www.sraoss.co.jp</a><br>
&gt;&gt;&gt;<br>
&gt;&gt;<br>
</blockquote></div></div>