<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, May 31, 2019 at 12:51 AM 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">&gt; Hi Ishii San<br>
&gt; <br>
&gt; <br>
&gt; <br>
&gt; On Thu, May 30, 2019 at 3:46 PM Tatsuo Ishii &lt;<a href="mailto:ishii@sraoss.co.jp" target="_blank">ishii@sraoss.co.jp</a>&gt; wrote:<br>
&gt; <br>
&gt;&gt; Usama,<br>
&gt;&gt;<br>
&gt;&gt; Are you going to add new parameter for this?  I vaguely recall that we<br>
&gt;&gt; agreed to add a new parameter for this in a telephone discussion.<br>
&gt;&gt;<br>
&gt; <br>
&gt; I am sorry somehow I might have missed that discussion and couldn&#39;t recall<br>
&gt; it :-(<br>
&gt; <br>
&gt; Are you of the point of view that we should make the behavior optional by<br>
&gt; adding<br>
&gt; a config parameter to control it?<br>
<br>
Yes, I am. The change is rather big one, and it&#39;d be best to give a<br>
chance to keep the existing behavior or not.<br></blockquote><div><br></div><div>Sure, I second you on that, it&#39;s a big behavioral change and must be configurable.</div><div><br></div><div>I will draft and share a patch for that.</div><div><br></div><div>Thanks</div><div>Best Regards</div><div>Muhammad Usama</div><div><br></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; Thanks<br>
&gt; Best regards<br>
&gt; Muhammad Usama<br>
&gt; <br>
&gt; <br>
&gt; <br>
&gt;&gt;<br>
&gt;&gt; Best regards,<br>
&gt;&gt; --<br>
&gt;&gt; Tatsuo Ishii<br>
&gt;&gt; SRA OSS, Inc. Japan<br>
&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; Japanese:<a href="http://www.sraoss.co.jp" rel="noreferrer" target="_blank">http://www.sraoss.co.jp</a><br>
&gt;&gt;<br>
&gt;&gt; From: Muhammad Usama &lt;<a href="mailto:m.usama@gmail.com" target="_blank">m.usama@gmail.com</a>&gt;<br>
&gt;&gt; Subject: [pgpool-committers: 5790] pgpool: Second part for<br>
&gt;&gt; [pgpool-hackers: 3295] duplicate failover reque<br>
&gt;&gt; Date: Sun, 26 May 2019 21:37:58 +0000<br>
&gt;&gt; Message-ID: &lt;<a href="mailto:E1hV0q6-0000rS-OH@gothos.postgresql.org" target="_blank">E1hV0q6-0000rS-OH@gothos.postgresql.org</a>&gt;<br>
&gt;&gt;<br>
&gt;&gt; &gt; Second part for [pgpool-hackers: 3295] duplicate failover request ...fix<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; As per the discussion on the thread [pgpool-hackers: 3295] we came to the<br>
&gt;&gt; &gt; conclusion, that the master watchdog node should resign from master<br>
&gt;&gt; &gt; responsibilities if the primary backend node gets into quarantine state<br>
&gt;&gt; on that.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; The commit implements the said behaviour by making the<br>
&gt;&gt; master/coordinator watchdog<br>
&gt;&gt; &gt; node resign from its status if it fails to get the consensus for the<br>
&gt;&gt; quarantined<br>
&gt;&gt; &gt; primary node failover, with in FAILOVER_COMMAND_FINISH_TIMEOUT(15)<br>
&gt;&gt; seconds.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; When the watchdog master resigns, because of quarantined primary node its<br>
&gt;&gt; &gt; wd_priority is decreased to (-1), so that it should get the least<br>
&gt;&gt; preference<br>
&gt;&gt; &gt; in the next election for the master/coordinator node selection. And once<br>
&gt;&gt; the<br>
&gt;&gt; &gt; election is concluded the wd_priority for the node gets restored to the<br>
&gt;&gt; &gt; original configured value.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; In case of failed consensus for standby node failover no action is taken.<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Branch<br>
&gt;&gt; &gt; ------<br>
&gt;&gt; &gt; master<br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Details<br>
&gt;&gt; &gt; -------<br>
&gt;&gt; &gt;<br>
&gt;&gt; <a href="https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=33df0d33df1ce701f07fecaeef5b87a2707c08f2" rel="noreferrer" target="_blank">https://git.postgresql.org/gitweb?p=pgpool2.git;a=commitdiff;h=33df0d33df1ce701f07fecaeef5b87a2707c08f2</a><br>
&gt;&gt; &gt;<br>
&gt;&gt; &gt; Modified Files<br>
&gt;&gt; &gt; --------------<br>
&gt;&gt; &gt; src/watchdog/watchdog.c | 47<br>
&gt;&gt; +++++++++++++++++++++++++++++++++++++++++++++--<br>
&gt;&gt; &gt; 1 file changed, 45 insertions(+), 2 deletions(-)<br>
&gt;&gt; &gt;<br>
&gt;&gt;<br>
</blockquote></div></div>