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

Sandy Ryza commented on YARN-184:
---------------------------------

Putting the QueueManager assignment in reinitialize would either require a 
findbugs exclude or synchronizing all accesses to it, which seems unnecessary 
because its value never changes after creation.

I'll update the comment. [~tucu00] is there anything else you'd like me to 
change?
                
> Remove unnecessary locking in fair scheduler, and address findbugs excludes.
> ----------------------------------------------------------------------------
>
>                 Key: YARN-184
>                 URL: https://issues.apache.org/jira/browse/YARN-184
>             Project: Hadoop YARN
>          Issue Type: Improvement
>    Affects Versions: 2.0.2-alpha
>            Reporter: Sandy Ryza
>            Assignee: Sandy Ryza
>         Attachments: YARN-184-1.patch, YARN-184-2.patch, YARN-184.patch
>
>
> In YARN-12, locks were added to all fields of QueueManager to address 
> findbugs.  In addition, findbugs exclusions were added in response to 
> MAPREDUCE-4439, without a deep look at the code.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to