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

Sunil G commented on YARN-6872:
-------------------------------

bq.isRecovery flag is already passed into 
SchedulerUtils#normalizeAndValidateRequest, I think we can use that flag 
directly ?
In {{normalizeAndValidateRequest}}, we check for the node label disabled check 
before looking for isRecovery flag. We could do some optimization here, but its 
a public api. I will check all references.

bq.And this block of code can be removed now ?
This could be removed. Just a thought ,that log is helpful rt. If we need that 
log, i guess some checks are needed there.

> Ensure apps could run given NodeLabels are disabled post RM switchover/restart
> ------------------------------------------------------------------------------
>
>                 Key: YARN-6872
>                 URL: https://issues.apache.org/jira/browse/YARN-6872
>             Project: Hadoop YARN
>          Issue Type: Bug
>          Components: resourcemanager
>            Reporter: Sunil G
>            Assignee: Sunil G
>         Attachments: YARN-6872.001.patch
>
>
> Post YARN-6031, few apps could be failed during recovery provided they had 
> some label requirements for AM and labels were disable post RM 
> restart/switchover. As discussed in YARN-6031, its better to run such apps as 
> it may be long running apps as well.



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