<div dir="ltr"><div dir="ltr"></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 2, 2019 at 5:45 PM Atin Mukherjee &lt;<a href="mailto:atin.mukherjee83@gmail.com">atin.mukherjee83@gmail.com</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"><div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 2 May 2019 at 20:38, Xavi Hernandez &lt;<a href="mailto:xhernandez@redhat.com" target="_blank">xhernandez@redhat.com</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"><div dir="ltr"></div><div dir="ltr"><div dir="ltr">On Thu, May 2, 2019 at 4:06 PM Atin Mukherjee &lt;<a href="mailto:atin.mukherjee83@gmail.com" target="_blank">atin.mukherjee83@gmail.com</a>&gt; wrote:<br></div></div><div dir="ltr"><div class="gmail_quote"></div></div><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br></div><div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 2 May 2019 at 19:14, Xavi Hernandez &lt;<a href="mailto:xhernandez@redhat.com" target="_blank">xhernandez@redhat.com</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"><div dir="auto"><div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 2 May 2019, 15:37 Milind Changire, &lt;<a href="mailto:mchangir@redhat.com" target="_blank">mchangir@redhat.com</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"><div dir="ltr">On Thu, May 2, 2019 at 6:44 PM Xavi Hernandez &lt;<a href="mailto:xhernandez@redhat.com" rel="noreferrer" target="_blank">xhernandez@redhat.com</a>&gt; wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div dir="ltr">Hi Ashish,<br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, May 2, 2019 at 2:17 PM Ashish Pandey &lt;<a href="mailto:aspandey@redhat.com" rel="noreferrer" target="_blank">aspandey@redhat.com</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"><div><div style="font-family:&quot;times new roman&quot;,&quot;new york&quot;,times,serif;font-size:12pt;color:rgb(0,0,0)"><div>Xavi,<br></div><div><br></div><div>I would like to keep this option (features.lock-notify-contention) enabled by default.<br></div><div>However, I can see that there is one more option which will impact the working of this option which is &quot;notify-contention-delay&quot;<br></div></div></div></blockquote></div></div></blockquote><div><br></div><div>Just a nit. I wish the option was called &quot;notify-contention-interval&quot;</div><div>The &quot;delay&quot; part doesn&#39;t really emphasize where the delay would be put in.<br></div></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">It makes sense. Maybe we can also rename it or add a second name (alias). If there are no objections, I will send a patch with the change.</div></div><div dir="auto"><div dir="auto"><br></div><div dir="auto">Xavi</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></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"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="font-family:&quot;times new roman&quot;,&quot;new york&quot;,times,serif;font-size:12pt;color:rgb(0,0,0)"><div></div><div>     .description = &quot;This value determines the minimum amount of time &quot;<br>                    &quot;(in seconds) between upcall contention notifications &quot;<br>                    &quot;on the same inode. If multiple lock requests are &quot;<br>                    &quot;received during this period, only one upcall will &quot;<br>                    &quot;be sent.&quot;},<br><br></div><div>I am not sure what should be the best value for this option if we want to keep features.lock-notify-contention ON by default?<br></div><div>It looks like if we keep the value of notify-contention-delay more, say 5 sec, it will wait for this much time to send up call<br></div><div>notification which does not look good.<br></div></div></div></blockquote><div><br></div><div>No, the first notification is sent immediately. What this option does is to define the minimum interval between notifications. This interval is per lock. This is done to avoid storms of notifications if many requests come referencing the same lock.</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"><div><div style="font-family:&quot;times new roman&quot;,&quot;new york&quot;,times,serif;font-size:12pt;color:rgb(0,0,0)"><div></div><div>Is my understanding correct?</div><div>What will be impact of this value and what should be the default value of this option?<br></div></div></div></blockquote><div><br></div><div>I think the current default value of 5 seconds seems good enough. If there are many bricks, each brick could send a notification per lock. 1000 bricks would mean a client would receive 1000 notifications every 5 seconds. It doesn&#39;t seem too much, but in those cases 10, and considering we could have other locks, maybe a higher value could be better.</div><div><br></div><div>Xavi</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"><div><div style="font-family:&quot;times new roman&quot;,&quot;new york&quot;,times,serif;font-size:12pt;color:rgb(0,0,0)"><div></div><div><br></div><div>---<br></div><div>Ashish<br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><div><br></div><hr id="gmail-m_-7791343529148458940m_-3012103280836931574gmail-m_48432577462973682m_1985893920105394902m_2194616033416799983gmail-m_-3673844415177098620gmail-m_-1192786914253342210zwchr"><div style="color:rgb(0,0,0);font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><b>From: </b>&quot;Xavi Hernandez&quot; &lt;<a href="mailto:xhernandez@redhat.com" rel="noreferrer" target="_blank">xhernandez@redhat.com</a>&gt;<br><b>To: </b>&quot;gluster-devel&quot; &lt;<a href="mailto:gluster-devel@gluster.org" rel="noreferrer" target="_blank">gluster-devel@gluster.org</a>&gt;<br><b>Cc: </b>&quot;Pranith Kumar Karampuri&quot; &lt;<a href="mailto:pkarampu@redhat.com" rel="noreferrer" target="_blank">pkarampu@redhat.com</a>&gt;, &quot;Ashish Pandey&quot; &lt;<a href="mailto:aspandey@redhat.com" rel="noreferrer" target="_blank">aspandey@redhat.com</a>&gt;, &quot;Amar Tumballi&quot; &lt;<a href="mailto:atumball@redhat.com" rel="noreferrer" target="_blank">atumball@redhat.com</a>&gt;<br><b>Sent: </b>Thursday, May 2, 2019 4:15:38 PM<br><b>Subject: </b>Should we enable contention notification by default ?<br><div><br></div><div dir="ltr">Hi all,<div><br></div><div>there&#39;s a feature in the locks xlator that sends a notification to current owner of a lock when another client tries to acquire the same lock. This way the current owner is made aware of the contention and can release the lock as soon as possible to allow the other client to proceed.</div><div><br></div><div>This is specially useful when eager-locking is used and multiple clients access the same files and directories. Currently both replicated and dispersed volumes use eager-locking and can use contention notification to force an early release of the lock.</div><div><br></div><div>Eager-locking reduces the number of network requests required for each operation, improving performance, but could add delays to other clients while it keeps the inode or entry locked. With the contention notification feature we avoid this delay, so we get the best performance with minimal issues in multiclient environments.</div><div><br></div><div>Currently the contention notification feature is controlled by the &#39;features.lock-notify-contention&#39; option and it&#39;s disabled by default. Should we enable it by default ?<br></div><div><br></div><div>I don&#39;t see any reason to keep it disabled by default. Does anyone foresee any problem ?</div></div></div></div></div></blockquote></div></div></blockquote></div></div></blockquote></div></div></div></blockquote><div dir="auto"><br></div><div dir="auto">Is it a server only option? Otherwise it will break backward compatibility if we rename the key, If alias can get this fixed, that’s a better choice but I’m not sure if it solves all the problems.</div></div></div></blockquote><div><br></div></div></div><div dir="ltr"><div class="gmail_quote"><div>It&#39;s a server side option. I though that an alias didn&#39;t have any other implication than accept two names for the same option. Is there anything else I need to consider ?</div></div></div></blockquote><div dir="auto"><br></div><div dir="auto">If it’s a server side option then there’s no challenge in alias. If you do rename then in heterogeneous server versions volume set wouldn’t work though.</div></div></div></blockquote><div><br></div><div>I created a patch to change this and set notify-contention to &#39;yes&#39; by default. I&#39;ll test upgrade paths to make sure that nothing breaks.</div><div><br></div><div>Xavi</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"><div><div class="gmail_quote"><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div></div></div></div><div dir="ltr"><div class="gmail_quote"><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"><div><div class="gmail_quote"><div dir="auto"><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><div style="font-family:&quot;times new roman&quot;,&quot;new york&quot;,times,serif;font-size:12pt;color:rgb(0,0,0)"><div style="color:rgb(0,0,0);font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><div dir="ltr"><div></div><div><br></div><div>Regards,</div><div><br></div><div>Xavi</div></div></div><div><br></div></div></div></blockquote></div></div>
_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" rel="noreferrer" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="https://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer noreferrer" target="_blank">https://lists.gluster.org/mailman/listinfo/gluster-devel</a></blockquote></div><br clear="all"><br>-- <br><div dir="ltr" class="gmail-m_-7791343529148458940m_-3012103280836931574gmail-m_48432577462973682m_1985893920105394902m_2194616033416799983gmail_signature"><div dir="ltr"><div><div dir="ltr">Milind<br><br></div></div></div></div></div>
</blockquote></div></div></div>
_______________________________________________<br>
Gluster-devel mailing list<br>
<a href="mailto:Gluster-devel@gluster.org" target="_blank">Gluster-devel@gluster.org</a><br>
<a href="https://lists.gluster.org/mailman/listinfo/gluster-devel" rel="noreferrer" target="_blank">https://lists.gluster.org/mailman/listinfo/gluster-devel</a></blockquote></div></div>-- <br><div dir="ltr" class="gmail-m_-7791343529148458940m_-3012103280836931574gmail-m_48432577462973682gmail_signature">--Atin</div>
</blockquote></div></div>
</blockquote></div></div>-- <br><div dir="ltr" class="gmail-m_-7791343529148458940gmail_signature">--Atin</div>
</blockquote></div></div>