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

Siddharth Seth commented on YARN-127:
-------------------------------------

Couple of comments.
 - Should AdminSecurityInfo move yarn-common ?
 - RMAdminProtocolPB.java sets the protocol name to  
"org.apache.hadoop.yarn.server.nodemanager.api.RMAdminProtocolPB". The 
nodemanager part doesn't make a lot of sense. This exists in the current code 
as well - so this could be fixed in a separate jira. In fact all Yarn protocols 
are named *PB (ClientRMProtocolPB, RMAdminProtocolPB, etc). The PB bit isn't 
required.
                
> Move RMAdmin tool to the client package
> ---------------------------------------
>
>                 Key: YARN-127
>                 URL: https://issues.apache.org/jira/browse/YARN-127
>             Project: Hadoop YARN
>          Issue Type: Bug
>            Reporter: Vinod Kumar Vavilapalli
>            Assignee: Vinod Kumar Vavilapalli
>         Attachments: YARN-127-20120924.txt
>
>
> It belongs to the client package and not the RM clearly.

--
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