<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Apr 19, 2016 at 5:30 AM, Lindsay Mathieson <span dir="ltr">&lt;<a href="mailto:lindsay.mathieson@gmail.com" target="_blank">lindsay.mathieson@gmail.com</a>&gt;</span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
  

    
  
  <div bgcolor="#FFFFFF" text="#000000">
    Is it possible to confirm the following understandings on my part?<br>
    <br>
    <br>
    - heal info shows the list of files with uncommitted writes across
    the bricks that would need heals <b><span>*</span>if<span>*</span></b>
    a brick went down before the write was committed.
    <br></div></blockquote><div><br></div><div>Yes and no. So heal-info reports both files that truly need heal and those that are currently participating in an uncommitted transaction. The latter entries are reported as &quot;possible undergoing heal&quot;. I believe we are going to drop the latter category of entries going forward. IIUC Pranith is working on something that does that. But I will let him confirm that.<br><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>
      * additionally it lists files being healed if a heal is in process
    <br>
      * writes can be data and metadata.
    <br> <br></div></blockquote><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    <br>
    - statics heal-count shows the count of files <b><span>*</span>actually
      needing healing<span>*</span></b></div></blockquote><div><br></div><div>This is correct for the most part. The only remote case where it will not report pending heals when it is supposed to is if some entries were marked &quot;dirty&quot; (indicated by the presence of their index under .glusterfs/indices/dirty). So it only considers count of entries needing heal in .glusterfs/indices/xattrop but not .glusterfs/indices/dirty.<br><br></div><div>For what it&#39;s worth, I realised that we don&#39;t really need to use locks to examine whether files need heal or not if the entries are under indices/xattrop. So if there is an entry under .glusterfs/indices/xattrop, it means it definitely needs heal. Only the entries under .glusterfs/indices/dirty need careful inspection. This will significantly enhance heal-info response time. So I will<br></div><div>work on getting this change in, maybe for the next .x release.<br></div><div><br></div><div>-Krutika<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div bgcolor="#FFFFFF" text="#000000">
    <br>
    <br>
    <br>
    <br>
    <br>
    So its perfectly ok to see files being listed by heal info on a busy
    cluster, so long as they aren&#39;t marked as a heal in progress.
    <br>
    <br>
    <br>
    thanks ,<span class="HOEnZb"><font color="#888888"><br>
    <pre cols="72">-- 
Lindsay Mathieson</pre>
  </font></span></div>

<br>_______________________________________________<br>
Gluster-users mailing list<br>
<a href="mailto:Gluster-users@gluster.org">Gluster-users@gluster.org</a><br>
<a href="http://www.gluster.org/mailman/listinfo/gluster-users" rel="noreferrer" target="_blank">http://www.gluster.org/mailman/listinfo/gluster-users</a><br></blockquote></div><br></div></div>