[ https://issues.apache.org/jira/browse/YARN-5921?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15705599#comment-15705599 ]
Rohith Sharma K S commented on YARN-5921: ----------------------------------------- oh.. that's fine. I am not sure about impact of using --amend. May be some kind of documentation for committers like FAQ would be good in wiki. > Incorrect synchronization in RMContextImpl#setHAServiceState/getHAServiceState > ------------------------------------------------------------------------------ > > Key: YARN-5921 > URL: https://issues.apache.org/jira/browse/YARN-5921 > Project: Hadoop YARN > Issue Type: Bug > Reporter: Varun Saxena > Assignee: Varun Saxena > Attachments: YARN-5921.01.patch, YARN-5921.02.patch > > > Code in RMContextImpl is as under: > {code:title=RMContextImpl.java|borderStyle=solid} > void setHAServiceState(HAServiceState haServiceState) { > synchronized (haServiceState) { > this.haServiceState = haServiceState; > } > } > public HAServiceState getHAServiceState() { > synchronized (haServiceState) { > return haServiceState; > } > } > {code} > As can be seen above, in setHAServiceState, we are synchronizing on the > passed haServiceState instead of haServiceState in RMContextImpl which will > not lead to desired effect. This does not seem to be intentional. > We can use a RW lock or synchronize on some object here. -- This message was sent by Atlassian JIRA (v6.3.4#6332) --------------------------------------------------------------------- To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org