[GitHub] zookeeper pull request #650: ZOOKEEPER-1908: setAcl should be have a recursi...

[lvfangmin]

Github user lvfangmin commented on a diff in the pull request:

    https://github.com/apache/zookeeper/pull/650#discussion_r222432633
  
    --- Diff: src/java/main/org/apache/zookeeper/cli/SetAclCommand.java ---
    @@ -19,12 +19,17 @@
     
     import java.util.List;
     import org.apache.commons.cli.*;
    +import org.apache.zookeeper.AsyncCallback.StringCallback;
     import org.apache.zookeeper.KeeperException;
    +import org.apache.zookeeper.ZKUtil;
     import org.apache.zookeeper.data.ACL;
     import org.apache.zookeeper.data.Stat;
     
     /**
    - * setAcl command for cli
    + * setAcl command for cli.
    + * Available options are s for printing znode's stats, v for set version 
of znode(s), R for
    + * recursive setting. User can combine v and R options together, but not s 
and R considering the
    + * number of znodes could be large.
    --- End diff --
    
    Thanks for adding the description for this.


---