hadoop-yarn-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Rohith Sharma K S (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (YARN-6211) Synchronization improvement in move and priority
Date Wed, 22 Feb 2017 05:29:44 GMT

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

Rohith Sharma K S commented on YARN-6211:
-----------------------------------------

Thanks Bibin for the patch.. One comment
# For both methods in RMAppManager i.e moveApplicationAcrossQueue and updateApplicationPriority,
pass the RMApp reference which has been validated by clientRMService. Do not again get it
from context in RMAppManager. It would lead to NPE in very corner cases. 
# Use the RMApp reference to get applicationId in synchronized block. See updateApplicationTimeout
method for reference.  

> Synchronization improvement in move and priority
> ------------------------------------------------
>
>                 Key: YARN-6211
>                 URL: https://issues.apache.org/jira/browse/YARN-6211
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Bibin A Chundatt
>            Assignee: Bibin A Chundatt
>         Attachments: YARN-6211.001.patch
>
>
> Application appid is wrongly taken for synchronization



--
This message was sent by Atlassian JIRA
(v6.3.15#6346)

---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscribe@hadoop.apache.org
For additional commands, e-mail: yarn-issues-help@hadoop.apache.org


Mime
View raw message