[
https://issues.apache.org/jira/browse/YARN-6069?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15869865#comment-15869865
]
Varun Saxena commented on YARN-6069:
------------------------------------
Now coming to the patch, the thing we have to consider is that we already have
yarn timeline-service related CORS configurations. So if somebody looks at it,
they may think this has to be used for ATSv2 too. Please note that we reuse
most of the existing ATSv1 configurations for ATSv2 too (wherever possible).
However, as ATSv2 is a new system, so we can decide what to use. We can use
either old timeline service configurations(as we did in 1st patch) or reuse
existing http CORS configurations (as in 2nd patch).
The advantage of first is that if we are using same config file for multiple
modules; and if we want to enable/disable CORS for ATS or use different set of
allowed headers, irrespective of what is configured for other modules, we can
do it.
This however is not a use case for us as we use distinct set of config files
for each module.
So I am fine with either.
I am -0 on what has been done in 2nd patch. Therefore, we can go with the
majority opinion here.
If majority wants to go with approach in 2nd patch, I would suggest to link
HTTP Authentication page's CORS section in timeline service HTML page telling
user to refer to other CORS related configurations in addition to enabling and
disabling it.
> CORS support in timeline v2
> ---------------------------
>
> Key: YARN-6069
> URL: https://issues.apache.org/jira/browse/YARN-6069
> Project: Hadoop YARN
> Issue Type: Sub-task
> Components: timelinereader
> Reporter: Sreenath Somarajapuram
> Assignee: Rohith Sharma K S
> Attachments: YARN-6069-YARN-5355.0001.patch,
> YARN-6069-YARN-5355.0002.patch
>
>
> By default the browser prevents accessing resources from multiple domains. In
> most cases the UIs would be loaded form a domain different from that of
> timeline server. Hence without CORS support, it would be difficult for the
> UIs to load data from timeline v2.
> YARN-2277 must provide more info on the implementation.
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
---------------------------------------------------------------------
To unsubscribe, e-mail: yarn-issues-unsubscr...@hadoop.apache.org
For additional commands, e-mail: yarn-issues-h...@hadoop.apache.org
