[
https://issues.apache.org/jira/browse/HADOOP-14543?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16057396#comment-16057396
]
Brahma Reddy Battula commented on HADOOP-14543:
-----------------------------------------------
bq.The Zookeeper#setACL() method sets the version number of the SetACLRequest.
So ideally, we should be setting the version only and not the aversion.
data and acls versions are modified independently.Here {{version}} is just an
argument which passed from client and added in request.
[Java doc here
|https://github.com/apache/zookeeper/blob/master/src/java/main/org/apache/zookeeper/ZooKeeper.java#L2368]
{noformat}
* 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
* node.
{noformat}
> Should use getAversion() while setting the zkacl
> ------------------------------------------------
>
> Key: HADOOP-14543
> URL: https://issues.apache.org/jira/browse/HADOOP-14543
> Project: Hadoop Common
> Issue Type: Bug
> Reporter: Brahma Reddy Battula
> Assignee: Brahma Reddy Battula
> Attachments: HADOOP-14543.patch
>
>
> while setting the zkacl we used {color:red}{{getVersion()}}{color} which is
> dataVersion,Ideally we should use {{{color:#14892c}getAversion{color}()}}. If
> there is any acl changes( i.e relam change/..) ,we set the ACL with
> dataversion which will cause {color:#d04437}BADVersion {color}and
> {color:#d04437}*process will not start*{color}. See
> [here|https://issues.apache.org/jira/browse/HDFS-11403?focusedCommentId=16051804&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16051804]
> {{zkClient.setACL(path, zkAcl, stat.getVersion());}}
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: common-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: common-issues-h...@hadoop.apache.org
