diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterStatus.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterStatus.java index d12ad0db908..c116bb45251 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterStatus.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/ClusterStatus.java @@ -27,11 +27,9 @@ import java.util.Map; import org.apache.hadoop.hbase.classification.InterfaceAudience; import org.apache.hadoop.hbase.master.RegionState; -import org.apache.hadoop.io.VersionedWritable; import com.google.common.base.Objects; - /** * Status information on the HBase cluster. *

@@ -70,17 +68,9 @@ import com.google.common.base.Objects; * */ @InterfaceAudience.Public -public class ClusterStatus extends VersionedWritable { - /** - * Version for object serialization. Incremented for changes in serialized - * representation. - *

- *
0
Initial version
- *
1
Added cluster ID
- *
2
Added Map of ServerName to ServerLoad
- *
3
Added master and backupMasters
- *
- */ +public class ClusterStatus { + + // TODO: remove this in 3.0 private static final byte VERSION = 2; private String hbaseVersion; @@ -209,8 +199,7 @@ public class ClusterStatus extends VersionedWritable { } ClusterStatus other = (ClusterStatus) o; //TODO Override the equals() methods in ServerLoad. - return (getVersion() == other.getVersion()) && - Objects.equal(getHBaseVersion(), other.getHBaseVersion()) && + return Objects.equal(getHBaseVersion(), other.getHBaseVersion()) && Objects.equal(this.liveServers, other.liveServers) && getDeadServerNames().containsAll(other.getDeadServerNames()) && Arrays.equals(getMasterCoprocessors(), other.getMasterCoprocessors()) && @@ -222,11 +211,16 @@ public class ClusterStatus extends VersionedWritable { * @see java.lang.Object#hashCode() */ public int hashCode() { - return VERSION + Objects.hashCode(hbaseVersion, liveServers, deadServers, + return Objects.hashCode(hbaseVersion, liveServers, deadServers, master, backupMasters); } - /** @return the object version number */ + /** + * + * @return the object version number + * @deprecated As of release 2.0.0, this will be removed in HBase 3.0.0 + */ + @Deprecated public byte getVersion() { return VERSION; } @@ -369,6 +363,7 @@ public class ClusterStatus extends VersionedWritable { /** * Builder for construct a ClusterStatus. */ + @InterfaceAudience.Private public static class Builder { private String hbaseVersion = null; private Map liveServers = null;