[ https://issues.apache.org/jira/browse/ZOOKEEPER-2818?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16070797#comment-16070797 ]
ASF GitHub Bot commented on ZOOKEEPER-2818: ------------------------------------------- Github user hanm commented on a diff in the pull request: https://github.com/apache/zookeeper/pull/291#discussion_r125140546 --- Diff: src/java/main/org/apache/zookeeper/ZooKeeper.java --- @@ -2347,25 +2347,25 @@ public void getACL(final String path, Stat stat, ACLCallback cb, /** * Set the ACL for the node of the given path if such a node exists and the - * given version matches the version of the node. Return the stat of the + * given aclVersion matches the aclVersion of the node. Return the stat of the * node. * <p> * A KeeperException with error code KeeperException.NoNode will be thrown * if no node with the given path exists. * <p> * A KeeperException with error code KeeperException.BadVersion will be - * thrown if the given version does not match the node's version. + * thrown if the given aclVersion does not match the node's aclVersion. * * @param path * @param acl - * @param version + * @param aclVersion --- End diff -- Thanks for updating. Yes no need patch every parameter in this patch. >> given aclVersion matches the aclVersion of the node. How about this: `given aclVersion matches the acl version of the node`. Similarly for the param: >> aclVersion the given aclVersion for the node `aclVersion - the given acl version of the node`. Copy pasting does not make it more descriptive, thus my suggestion. > Improve the ZooKeeper#setACL java doc > -------------------------------------- > > Key: ZOOKEEPER-2818 > URL: https://issues.apache.org/jira/browse/ZOOKEEPER-2818 > Project: ZooKeeper > Issue Type: Bug > Reporter: Brahma Reddy Battula > > As per discussion in > [mailinglist|http://mail-archives.apache.org/mod_mbox/zookeeper-user/201706.mbox/browser],It's > better improve Java doc or argument which might not mislead for new users. -- This message was sent by Atlassian JIRA (v6.4.14#64029)