[
https://issues.apache.org/jira/browse/YARN-1277?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13787490#comment-13787490
]
Vinod Kumar Vavilapalli commented on YARN-1277:
-----------------------------------------------
I'm happy I spent time pushing for polished patches previously at YARN-1280.
The code is so much cleaner making it to easy to accommodate changes from this
patch.
Patch looks good overall. Quick comments:
- We aren't removing hadoop.ssl.enabled yet?
- Why two methods getResolvedRMWebAppURLWithoutScheme* in YARN's
WebAppUtils.java? Looks like only one is enough.
- AmFilterInitializer changes are import only and can be skipped.
- Earlier we punted on enabling https for proxy-server, but it should be easy
to add now?
> Add http policy support for YARN daemons
> ----------------------------------------
>
> Key: YARN-1277
> URL: https://issues.apache.org/jira/browse/YARN-1277
> Project: Hadoop YARN
> Issue Type: Sub-task
> Affects Versions: 2.0.0-alpha
> Reporter: Suresh Srinivas
> Assignee: Omkar Vinit Joshi
> Attachments: YARN-1277.20131005.1.patch, YARN-1277.20131005.2.patch,
> YARN-1277.patch
>
>
> This YARN part of HADOOP-10022.
--
This message was sent by Atlassian JIRA
(v6.1#6144)
