[ 
https://issues.apache.org/jira/browse/YARN-10877?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Szilard Nemeth reassigned YARN-10877:
-------------------------------------

    Assignee:     (was: Szilard Nemeth)

> SLSSchedulerCommons: Consider using application map from 
> AbstractYarnScheduler and make event handling more consistent
> ----------------------------------------------------------------------------------------------------------------------
>
>                 Key: YARN-10877
>                 URL: https://issues.apache.org/jira/browse/YARN-10877
>             Project: Hadoop YARN
>          Issue Type: Improvement
>            Reporter: Szilard Nemeth
>            Priority: Minor
>
> This is a follow-up of YARN-10552.
> The improvements and things to check are coming from [this 
> comment|https://issues.apache.org/jira/browse/YARN-10552?focusedCommentId=17277991&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-17277991].
> {quote}
> appQueueMap was not present in SLSFairScheduler before (it was in 
> SLSCapacityScheduler) however from 
> https://github.com/apache/hadoop/blob/trunk/hadoop-tools/hadoop-sls/src/main/java/org/apache/hadoop/yarn/sls/scheduler/SLSFairScheduler.java#L163,
>  it seems that the super class of the schedulers - 
> https://github.com/apache/hadoop/blob/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-server/hadoop-yarn-server-resourcemanager/src/main/java/org/apache/hadoop/yarn/server/resourcemanager/scheduler/AbstractYarnScheduler.java#L159
>  has this already. As such, do we really need to define a new map as a common 
> map at all in SLSSchedulerCommons or can we somehow reuse the super class's 
> map? It might need some code updates though.
> In regards to the above point, considering SLSFairScheduler did not 
> previously have any of the following code in handle() method:
> {quote}



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

---------------------------------------------------------------------
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