hadoop-common-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Uma Maheswara Rao G (Commented) (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (HADOOP-7925) Add interface and update CLI to query current state to HAServiceProtocol
Date Thu, 15 Dec 2011 02:23:30 GMT

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

Uma Maheswara Rao G commented on HADOOP-7925:

Eli, Patch looks good to me.

One small comment:
  Looks we are spreading the "active", "standby" string literals in common and hdfs. If some
one changes toString value of ActiveState in HDFS and prints the value somewhere that may
give the new value of state, at the same time we must change common also.
 My suggestion is to keep this literals in Enum itself and provide the toString of it. We
can use this in both the places now. This is just suggestion, what do you say? 
Also we need not have the state comparision checks.

enum HAServiceState {
      private String state = null;
      HAServiceState(String state)
        this.state = state;
      public String toString()
        return this.state;
> Add interface and update CLI to query current state to HAServiceProtocol
> ------------------------------------------------------------------------
>                 Key: HADOOP-7925
>                 URL: https://issues.apache.org/jira/browse/HADOOP-7925
>             Project: Hadoop Common
>          Issue Type: Sub-task
>    Affects Versions: HA Branch (HDFS-1623)
>            Reporter: Eli Collins
>            Assignee: Eli Collins
>         Attachments: hadoop-7925.txt, hadoop-7925.txt
> Common side of HDFS-2679.

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


View raw message