[Gluster-devel] Need inputs on patch #17985

[Ashish Pandey]

Raghvendra, 

I have provided my comment on this patch. 
I think EC will not have any issue with this approach. 
However, I would welcome comments from Xavi and Pranith too for any side effects which I may not be able to foresee. 

Ashish 

----- Original Message -----

From: "Raghavendra Gowdappa" <rgowdapp at redhat.com> 
To: "Ashish Pandey" <aspandey at redhat.com> 
Cc: "Pranith Kumar Karampuri" <pkarampu at redhat.com>, "Xavier Hernandez" <xhernandez at datalab.es>, "Gluster Devel" <gluster-devel at gluster.org> 
Sent: Wednesday, August 23, 2017 8:29:48 AM 
Subject: Need inputs on patch #17985 

Hi Ashish, 

Following are the blockers for making a decision on whether patch [1] can be merged or not: 
* Evaluation of dentry operations (like rename etc) in dht 
* 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 

Can you please comment on the patch? I'll take care of dht part. 

[1] https://review.gluster.org/#/c/17985/ 

regards, 
Raghavendra 

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.gluster.org/pipermail/gluster-devel/attachments/20170822/bbfd4063/attachment.html>