[jira] [Commented] (YARN-10856) Prevent ATS v2 health check REST API call if the ATS service itself is disabled.

["Szilard Nemeth (Jira)" <ji...@apache.org>]

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

Szilard Nemeth commented on YARN-10856:
---------------------------------------

Thanks [~sahuja] for filing this jira and thanks [~bteke] for working on this.
The code changes in the PR looks good to me, I just merged the PR to trunk.

> Prevent ATS v2 health check REST API call if the ATS service itself is disabled.
> --------------------------------------------------------------------------------
>
>                 Key: YARN-10856
>                 URL: https://issues.apache.org/jira/browse/YARN-10856
>             Project: Hadoop YARN
>          Issue Type: Improvement
>          Components: yarn-ui-v2
>    Affects Versions: 3.3.1
>            Reporter: Siddharth Ahuja
>            Assignee: Benjamin Teke
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 3.4.0
>
>          Time Spent: 0.5h
>  Remaining Estimate: 0h
>
> Currently, even if {{yarn.timeline-service.enabled}} is disabled, the UI2 code still goes ahead and performs timeline health check REST API calls, see [1], [2] and [3].
> This is un-necessary and can cause slowness issues with RM UI2 page loading if a firewall is dropping packets on the ATS v2 port (as it is not meant to be available e.g. 8188/8190) in the background and the timeout is not yet hit.
> This ATSv2 health check REST API call is redundant and should be prevented if the service itself is disabled.
> [1] https://github.com/apache/hadoop/blob/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/adapters/timeline-health.js
> [2] https://github.com/apache/hadoop/blob/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/routes/application.js#L34
> [3] https://github.com/apache/hadoop/blob/trunk/hadoop-yarn-project/hadoop-yarn/hadoop-yarn-ui/src/main/webapp/app/routes/yarn-app/logs.js#L40



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

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