<div dir="ltr"><div>Raghavendra,<br></div> As Ashish mentioned, there aren't any known problems if upper xlators don't send lookups in EC at the moment.<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Aug 23, 2017 at 9:07 AM, Ashish Pandey <span dir="ltr"><<a href="mailto:aspandey@redhat.com" target="_blank">aspandey@redhat.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div style="font-family:times new roman,new york,times,serif;font-size:12pt;color:#000000"><div>Raghvendra,<br></div><div><br></div><div>I have provided my comment on this patch.<br></div><div>I think EC will not have any issue with this approach. <br></div><div>However, I would welcome comments from Xavi and Pranith too for any side effects which I may not be able to foresee.<br></div><div><br></div><div>Ashish<br></div><div><br></div><hr id="m_375864711851788839zwchr"><div style="color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt"><b>From: </b>"Raghavendra Gowdappa" <<a href="mailto:rgowdapp@redhat.com" target="_blank">rgowdapp@redhat.com</a>><br><b>To: </b>"Ashish Pandey" <<a href="mailto:aspandey@redhat.com" target="_blank">aspandey@redhat.com</a>><br><b>Cc: </b>"Pranith Kumar Karampuri" <<a href="mailto:pkarampu@redhat.com" target="_blank">pkarampu@redhat.com</a>>, "Xavier Hernandez" <<a href="mailto:xhernandez@datalab.es" target="_blank">xhernandez@datalab.es</a>>, "Gluster Devel" <<a href="mailto:gluster-devel@gluster.org" target="_blank">gluster-devel@gluster.org</a>><br><b>Sent: </b>Wednesday, August 23, 2017 8:29:48 AM<br><b>Subject: </b>Need inputs on patch #17985<div><div class="h5"><br><div><br></div>Hi Ashish,<br><div><br></div>Following are the blockers for making a decision on whether patch [1] can be merged or not:<br>* Evaluation of dentry operations (like rename etc) in dht<br>* Whether EC works fine if a non-lookup fop (like open(dir), stat, chmod etc) hits EC without a single lookup performed on file/inode<br><div><br></div>Can you please comment on the patch? I'll take care of dht part.<br><div><br></div>[1] <a href="https://review.gluster.org/#/c/17985/" target="_blank">https://review.gluster.org/#/<wbr>c/17985/</a><br><div><br></div>regards,<br>Raghavendra<br></div></div></div><div><br></div></div></div></blockquote></div><br><br clear="all"><br>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">Pranith<br></div></div>
</div>