[
https://issues.apache.org/jira/browse/HADOOP-8805?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13457618#comment-13457618
]
Bo Wang commented on HADOOP-8805:
---------------------------------
I realized a problem that YARN RPC requires protocol implementation follows the
following naming convention: PROTOCOL + PBServiceImpl. This naming convention
is hard coded in
org.apache.hadoop.yarn.factories.impl.pb.RpcServerFactoryPBImpl#getPbServiceImplClassName.
Thus, to make GetUserMappingProtocol usable in both HDFS & YARN, we have to
follow the convention of YARN rather than HDFS..
In YARN, the protocbuf generated class shares the same name as the protocol
itself (e.g. RMAdminProtocol). Thus, the current patch follows YARN convention.
I have tested the patch from command line as follows.
(1) start a namenode
(2) bin/hdfs groups <username>
It returns the correct results.
> Move protocol buffer implementation of GetUserMappingProtocol from HDFS to
> Common
> ---------------------------------------------------------------------------------
>
> Key: HADOOP-8805
> URL: https://issues.apache.org/jira/browse/HADOOP-8805
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Bo Wang
> Assignee: Bo Wang
> Attachments: HADOOP-8805.patch, HADOOP-8805-v2.patch,
> HADOOP-8805-v3.patch
>
>
> org.apache.hadoop.tools.GetUserMappingProtocol is used in both HDFS and YARN.
> We should move the protocol buffer implementation from HDFS to Common so that
> it can also be used by YARN.
--
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
