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

Bhallamudi Venkata Siva Kamesh commented on MAPREDUCE-3354:
-----------------------------------------------------------

Jonathan,
minor comments from the patch. 

{noformat}usage="Usage: mapreduce-daemon.sh [--config <conf-dir>] [--hosts 
hostlistfile] (start|stop) <yarn-command> "{noformat}

I *think*, instead of mapreduce-daemon.sh, it should be mapred-daemon.sh in 
mapred-daemon.sh. Moreover after inclusion of this patch, it will be  nomore a 
yarn-command, it will be a mapred-command.

{noformat}$ $YARN_HOME/bin/mapred stop historyserver --config $HADOOP_CONF_DIR 
{noformat}
I *think*, we should replace HADOOP_CONF_DIR by YARN_CONF_DIR and mapred by 
mapred-daemon.sh in ClusterSetup.apt.vm.
                
> JobHistoryServer should be started by bin/mapred and not by bin/yarn
> --------------------------------------------------------------------
>
>                 Key: MAPREDUCE-3354
>                 URL: https://issues.apache.org/jira/browse/MAPREDUCE-3354
>             Project: Hadoop Map/Reduce
>          Issue Type: Bug
>          Components: jobhistoryserver, mrv2
>    Affects Versions: 0.23.0
>            Reporter: Vinod Kumar Vavilapalli
>            Assignee: Jonathan Eagles
>            Priority: Blocker
>         Attachments: MAPREDUCE-3354.patch, MAPREDUCE-3354.patch, 
> MAPREDUCE-3354.patch, MAPREDUCE-3354.patch, MAPREDUCE-3354.patch, 
> MAPREDUCE-3354.patch
>
>
> JobHistoryServer belongs to mapreduce land.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to