[
https://issues.apache.org/jira/browse/HADOOP-10043?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13794912#comment-13794912
]
Suresh Srinivas commented on HADOOP-10043:
------------------------------------------
I have not looked at the patch closely. [~ozawa], can you please add more
details about this. SecretManager and AbstractDelegationTokenSecretManager are
used in MapReduce/Yarn, HDFS and even in Hive. I am not convinced for YARN
related changes we need to make this change that impacts all the other users.
The main motivation seems to be - "Convert *SecretManagers in the RM to
services"
Why can't this be done using composition instead of inheritance?
> Convert org.apache.hadoop.security.token.SecretManager to be an
> AbstractService
> -------------------------------------------------------------------------------
>
> Key: HADOOP-10043
> URL: https://issues.apache.org/jira/browse/HADOOP-10043
> Project: Hadoop Common
> Issue Type: Improvement
> Reporter: Tsuyoshi OZAWA
> Assignee: Tsuyoshi OZAWA
> Attachments: HADOOP-10043.1.patch, HADOOP-10043.2.patch,
> HADOOP-10043.3.patch
>
>
> I'm dealing with YARN-1172, a subtask of YARN-1139(ResourceManager HA related
> task). The sentence as follows is a quoted from YARN-1172's my comment:
> {quote}
> I've found that it requires org.apache.hadoop.security.token.SecretManager to
> be an AbstractService,
> because both AbstractService and
> org.apache.hadoop.security.token.SecretManager are abstract class and we
> cannot extend both of them at the same time.
> {quote}
--
This message was sent by Atlassian JIRA
(v6.1#6144)
