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

Sunil G commented on YARN-4514:
-------------------------------

bq.tez-ui2, their configuration file is a configs.env instead of JS. which 
could be kept after build
They still have default-config.js. But main configs are in env. I will make 
these changes.

bq.Computed value is a good idea
This is a good option. Thanks for pointing out. Let me make changes in this 
line.


bq.And we may not need localBaseUrl, since it is a workaround before we fix 
CORS issues in YARN.
But I think still we need it for now as we are testing. I think in formal 
version, we can keep this config as empty. For now, we can keep a default 
value. Thoughts?

> [YARN-3368] Cleanup hardcoded configurations, such as RM/ATS addresses
> ----------------------------------------------------------------------
>
>                 Key: YARN-4514
>                 URL: https://issues.apache.org/jira/browse/YARN-4514
>             Project: Hadoop YARN
>          Issue Type: Sub-task
>            Reporter: Wangda Tan
>            Assignee: Sunil G
>         Attachments: YARN-4514-YARN-3368.1.patch, 
> YARN-4514-YARN-3368.2.patch, YARN-4514-YARN-3368.3.patch, 
> YARN-4514-YARN-3368.4.patch
>
>
> We have several configurations are hard-coded, for example, RM/ATS addresses, 
> we should make them configurable. 



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to