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

Zhijie Shen commented on YARN-1716:
-----------------------------------

Writer lock is not necessary in tranistion classes, all the transition code is 
already executed in the writer lock. See:
{code}
public void handle(RMContainerEvent event) {
{code}

> finishedStatus is accessed without proper locking in 
> RMContainerImpl#FinishedTransition#transition()
> ----------------------------------------------------------------------------------------------------
>
>                 Key: YARN-1716
>                 URL: https://issues.apache.org/jira/browse/YARN-1716
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Ted Yu
>         Attachments: YARN-1716.1.patch
>
>
> {code}
>       container.finishedStatus = finishedEvent.getRemoteContainerStatus();
> {code}
> The access should be guarded by obtaining readLock



--
This message was sent by Atlassian JIRA
(v6.1.5#6160)

Reply via email to