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

Junping Du commented on MAPREDUCE-4452:
---------------------------------------

Patrick, you are right that the threads of modifying and reading are pending on 
different lock object/attribute in my previous patch, so I update to v3 patch 
so both of them are synchronized on related attributes. The intention to get 
rid of synchronized getter methods (as v1 patch) is that will make QueueManager 
instance can only be accessed by one thread per time although different threads 
are reading/modifying different attributes. I guess that is the right way to do 
it. What do you think?
                
> Several Findbugs issues with new FairScheduler in YARN
> ------------------------------------------------------
>
>                 Key: MAPREDUCE-4452
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-4452
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: scheduler
>    Affects Versions: 2.1.0-alpha
>            Reporter: Junping Du
>            Assignee: Junping Du
>         Attachments: MAPREDUCE-4452-v1.patch, MAPREDUCE-4452-v2.patch, 
> MAPREDUCE-4452-v3.patch, MAPREDUCE-4452.patch
>
>
> The good feature of FairScheduler is added recently to YARN. As recently 
> PreCommit test from MAPREDUCE-4309, there are several bugs found by Findbugs 
> related to FairScheduler:
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FairSchedulerEventLog.shutdown()
>  might ignore java.lang.Exception
> Inconsistent synchronization of 
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.FairSchedulerEventLog.logDisabled;
>  locked 50% of time
> Inconsistent synchronization of 
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.QueueManager.queueMaxAppsDefault;
>  locked 50% of time
> Inconsistent synchronization of 
> org.apache.hadoop.yarn.server.resourcemanager.scheduler.fair.QueueManager.userMaxAppsDefault;
>  locked 50% of time
> The details are 
> in:https://builds.apache.org/job/PreCommit-MAPREDUCE-Build/2612//artifact/trunk/patchprocess/newPatchFindbugsWarningshadoop-yarn-server-resourcemanager.html#DE_MIGHT_IGNORE
>  

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to