[jira] [Commented] (HDFS-5614) NameNode: implement handling of ACLs in combination with snapshots.

[ https://issues.apache.org/jira/browse/HDFS-5614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13888077#comment-13888077 ] Jing Zhao commented on HDFS-5614: - The v2 patch looks great to me. Just some minors: # Not

[jira] [Commented] (HDFS-5614) NameNode: implement handling of ACLs in combination with snapshots.

[ https://issues.apache.org/jira/browse/HDFS-5614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13888222#comment-13888222 ] Jing Zhao commented on HDFS-5614: - bq. One slightly messy aspect of this change Yeah,

[jira] [Commented] (HDFS-5614) NameNode: implement handling of ACLs in combination with snapshots.

[ https://issues.apache.org/jira/browse/HDFS-5614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13888236#comment-13888236 ] Chris Nauroth commented on HDFS-5614: - bq. Or we can pass in a Feature.class array

[jira] [Commented] (HDFS-5614) NameNode: implement handling of ACLs in combination with snapshots.

[ https://issues.apache.org/jira/browse/HDFS-5614?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13886992#comment-13886992 ] Chris Nauroth commented on HDFS-5614: - I've just noticed that my v1 patch here has