<div dir="ltr"><div dir="ltr">Hi Ishii San<div><br></div><div><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 30, 2019 at 3:46 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">Usama,<br>
<br>
Are you going to add new parameter for this?  I vaguely recall that we<br>
agreed to add a new parameter for this in a telephone discussion.<br></blockquote><div><br></div><div>I am sorry somehow I might have missed that discussion and couldn&#39;t recall it :-(</div><div><br></div><div>Are you of the point of view that we should make the behavior optional by adding</div><div>a config parameter to control it? </div><div><br></div><div>Thanks</div><div>Best regards</div><div>Muhammad Usama</div><div><br></div><div> </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>
From: Muhammad Usama &lt;<a href="mailto:m.usama@gmail.com" target="_blank">m.usama@gmail.com</a>&gt;<br>
Subject: [pgpool-committers: 5790] pgpool: Second part for [pgpool-hackers: 3295] duplicate failover reque<br>
Date: Sun, 26 May 2019 21:37:58 +0000<br>
Message-ID: &lt;<a href="mailto:E1hV0q6-0000rS-OH@gothos.postgresql.org" target="_blank">E1hV0q6-0000rS-OH@gothos.postgresql.org</a>&gt;<br>
<br>
&gt; Second part for [pgpool-hackers: 3295] duplicate failover request ...fix<br>
&gt; <br>
&gt; As per the discussion on the thread [pgpool-hackers: 3295] we came to the<br>
&gt; conclusion, that the master watchdog node should resign from master<br>
&gt; responsibilities if the primary backend node gets into quarantine state on that.<br>
&gt; <br>
&gt; The commit implements the said behaviour by making the master/coordinator watchdog<br>
&gt; node resign from its status if it fails to get the consensus for the quarantined<br>
&gt; primary node failover, with in FAILOVER_COMMAND_FINISH_TIMEOUT(15) seconds.<br>
&gt; <br>
&gt; When the watchdog master resigns, because of quarantined primary node its<br>
&gt; wd_priority is decreased to (-1), so that it should get the least preference<br>
&gt; in the next election for the master/coordinator node selection. And once the<br>
&gt; election is concluded the wd_priority for the node gets restored to the<br>
&gt; original configured value.<br>
&gt; <br>
&gt; In case of failed consensus for standby node failover no action is taken.<br>
&gt; <br>
&gt; Branch<br>
&gt; ------<br>
&gt; master<br>
&gt; <br>
&gt; Details<br>
&gt; -------<br>
&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; <br>
&gt; Modified Files<br>
&gt; --------------<br>
&gt; src/watchdog/watchdog.c | 47 +++++++++++++++++++++++++++++++++++++++++++++--<br>
&gt; 1 file changed, 45 insertions(+), 2 deletions(-)<br>
&gt; <br>
</blockquote></div></div>