[
https://issues.apache.org/jira/browse/YARN-6211?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15877534#comment-15877534
]
Sunil G commented on YARN-6211:
-------------------------------
One more minor nit:
# There are unused imports in {{RMAppManager}}, please remove that also.
# Adding to [~rohithsharma] comments, in moveApplicationAcrossQueue and
updateApplicationPriority, appId from request itself could be passed. But you
can have null check for app, to ensure that app is not removed when call
reaches at either of these two methods.
> 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-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
