<html><body><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="zwchr"><div style="color:#000;font-weight:normal;font-style:normal;text-decoration:none;font-family:Helvetica,Arial,sans-serif;font-size:12pt;" data-mce-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" <rgowdapp@redhat.com><br><b>To: </b>"Ashish Pandey" <aspandey@redhat.com><br><b>Cc: </b>"Pranith Kumar Karampuri" <pkarampu@redhat.com>, "Xavier Hernandez" <xhernandez@datalab.es>, "Gluster Devel" <gluster-devel@gluster.org><br><b>Sent: </b>Wednesday, August 23, 2017 8:29:48 AM<br><b>Subject: </b>Need inputs on patch #17985<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] https://review.gluster.org/#/c/17985/<br><div><br></div>regards,<br>Raghavendra<br></div><div><br></div></div></body></html>