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

Omkar Vinit Joshi commented on YARN-1204:
-----------------------------------------

Thanks vinod for the review..

bq. Quickly looked at the patch. In ResourceManager.java and WebAppProxy.java, 
you replaced the usage of YarnConfiguration.getProxyHostAndPort() with 
WebAppUtils.getResolvedRMWebAppURL().
fixed..

bq. Can you also look at test-failures? That's a lot of them, some of them are 
obviously tracked elsewhere, let's make sure all test-failures are tracked.
I see all these test failures even without patch..
                
> Need to add https port related property in Yarn
> -----------------------------------------------
>
>                 Key: YARN-1204
>                 URL: https://issues.apache.org/jira/browse/YARN-1204
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Yesha Vora
>            Assignee: Omkar Vinit Joshi
>         Attachments: YARN-1204.20131018.1.patch, YARN-1204.20131020.1.patch, 
> YARN-1204.20131020.2.patch, YARN-1204.20131020.3.patch, 
> YARN-1204.20131020.4.patch, YARN-1204.20131023.1.patch
>
>
> There is no yarn property available to configure https port for Resource 
> manager, nodemanager and history server. Currently, Yarn services uses the 
> port defined for http [defined by 
> 'mapreduce.jobhistory.webapp.address','yarn.nodemanager.webapp.address', 
> 'yarn.resourcemanager.webapp.address'] for running services on https protocol.
> Yarn should have list of property to assign https port for RM, NM and JHS.
> It can be like below.
> yarn.nodemanager.webapp.https.address
> yarn.resourcemanager.webapp.https.address
> mapreduce.jobhistory.webapp.https.address 

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to