[ 
https://issues.apache.org/jira/browse/YARN-7053?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16135960#comment-16135960
 ] 

Jonathan Hung commented on YARN-7053:
-------------------------------------

Thanks [~subru] for the review. Added a ZKCuratorManager test.

Regarding ZookeeperFederationStateStore transactions, I didn't see any existing 
transactions in this class that could be refactored. And the only place adding 
transactions seemed applicable was in the {{put(String znode, byte[] data, 
boolean update)}} method. Did you want to change the {{put}} logic as part of 
this ticket?

> Move curator transaction support to ZKCuratorManager
> ----------------------------------------------------
>
>                 Key: YARN-7053
>                 URL: https://issues.apache.org/jira/browse/YARN-7053
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Jonathan Hung
>            Assignee: Jonathan Hung
>         Attachments: YARN-7053.001.patch, YARN-7053.002.patch, 
> YARN-7053.003.patch
>
>
> HADOOP-14741 moves curator functionality to ZKCuratorManager. ZKRMStateStore 
> has some curator transaction support which can be reused, so this can be 
> moved to ZKCuratorManager as well.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to