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

Xieming Li commented on HADOOP-16753:
-------------------------------------

[~aajisaka], Thank you for your comments.

Since we are currently using _hdfs haadmin -transtionTo\{Active, Standby, 
Observer}_ and _-faileover_ to manage HDFS states. Therefore, moving  
"-transitionToObserver,-failover" from HAAdmin.java to DFSAdmin.java will cause 
a situation where a superclass(HAAdmin) referencing the content in its 
subclass(DFSAdmin).

Thus, I think it might be a good idea to keep them as they are.

> Refactor HAAdmin
> ----------------
>
>                 Key: HADOOP-16753
>                 URL: https://issues.apache.org/jira/browse/HADOOP-16753
>             Project: Hadoop Common
>          Issue Type: Improvement
>            Reporter: Akira Ajisaka
>            Assignee: Xieming Li
>            Priority: Major
>         Attachments: HADOOP-16753.001.patch
>
>
> https://issues.apache.org/jira/browse/YARN-9985?focusedCommentId=16991414&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-16991414
> We should move HDFS-specific haadmin options from HAAdmin to DFSHAAdmin to 
> remove unnecessary if-else statements from RMAdmin command.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to