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

Zhijie Shen commented on YARN-1898:
-----------------------------------

LGTM. It should cover all the default apps/servlets in HttpServer2. One small 
nit:
{code}
+  private static final Set<String> nonRedirectedURIs = Sets.newHashSet("/conf",
+      "/stacks", "/logLevel", "/metrics", "/jmx", "/logs");
{code}
The constant should be NON_REDIRECTED_URIS by convention?

> Standby RM's conf, stack, logLevel and metrics links are redirecting to 
> Active RM
> ---------------------------------------------------------------------------------
>
>                 Key: YARN-1898
>                 URL: https://issues.apache.org/jira/browse/YARN-1898
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>          Components: resourcemanager
>            Reporter: Yesha Vora
>            Assignee: Xuan Gong
>         Attachments: YARN-1898.1.patch, YARN-1898.2.patch
>
>
> Standby RM links /conf, /stacks, /logLevel, /metrics, /jmx is redirected to 
> Active RM.
> It should not be redirected to Active RM



--
This message was sent by Atlassian JIRA
(v6.2#6252)

Reply via email to