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

Wilfred Spiegelenburg commented on YARN-7622:
---------------------------------------------

Thank you for the update.
You can not move the two values inside the thread because the method that you 
removed them from and which needs them is a public method which is called when 
you do a command line refresh of the configuration. The variables must be 
accessible from the reloader thread and from the {{reloadAllocations}} method.

You also have introduced a large number of new checkstyle issues most are 
unused imports please check those.
The junit test failures look unrelated but please confirm that they are.

> Allow fair-scheduler configuration on HDFS
> ------------------------------------------
>
>                 Key: YARN-7622
>                 URL: https://issues.apache.org/jira/browse/YARN-7622
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: fairscheduler, resourcemanager
>            Reporter: Greg Phillips
>            Assignee: Greg Phillips
>            Priority: Minor
>         Attachments: YARN-7622.001.patch, YARN-7622.002.patch
>
>
> The FairScheduler requires the allocation file to be hosted on the local 
> filesystem on the RM node(s). Allowing HDFS to store the allocation file will 
> provide improved redundancy, more options for scheduler updates, and RM 
> failover consistency in HA.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

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

Reply via email to