Skip to content

Commit 4c8c94a

Browse files
Rushabhbusbey
andcommitted
HBASE-22874 Define a public API for Canary checking and a non-public tool implementation
Closes #580 * Canary is now an IA.Public interface * CanaryTool is now the implementation Branch-1.3 specific changes for differences in APIs and cleanup of the ref guide using a classname. Co-authored-by: Sean Busbey <busbey@apache.org> Signed-off-by: Sean Busbey <busbey@apache.org>
1 parent 092b4b2 commit 4c8c94a

File tree

6 files changed

+1928
-1739
lines changed

6 files changed

+1928
-1739
lines changed

bin/hbase

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -400,7 +400,7 @@ elif [ "$COMMAND" = "ltt" ] ; then
400400
CLASS='org.apache.hadoop.hbase.util.LoadTestTool'
401401
HBASE_OPTS="$HBASE_OPTS $HBASE_LTT_OPTS"
402402
elif [ "$COMMAND" = "canary" ] ; then
403-
CLASS='org.apache.hadoop.hbase.tool.Canary'
403+
CLASS='org.apache.hadoop.hbase.tool.CanaryTool'
404404
HBASE_OPTS="$HBASE_OPTS $HBASE_CANARY_OPTS"
405405
elif [ "$COMMAND" = "hbtop" ] ; then
406406
CLASS='org.apache.hadoop.hbase.hbtop.HBTop'

hbase-server/src/main/jamon/org/apache/hadoop/hbase/tmpl/master/MasterStatusTmpl.jamon

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ org.apache.hadoop.hbase.master.RegionState;
4848
org.apache.hadoop.hbase.HTableDescriptor;
4949
org.apache.hadoop.hbase.HBaseConfiguration;
5050
org.apache.hadoop.hbase.TableName;
51-
org.apache.hadoop.hbase.tool.Canary;
51+
org.apache.hadoop.hbase.tool.CanaryTool;
5252
org.apache.hadoop.hbase.protobuf.generated.HBaseProtos.SnapshotDescription;
5353
org.apache.hadoop.hbase.master.DeadServer;
5454
org.apache.hadoop.hbase.protobuf.ProtobufUtil;
@@ -384,7 +384,7 @@ AssignmentManager assignmentManager = master.getAssignmentManager();
384384
<%java>String description = null;
385385
if (tableName.equals(TableName.META_TABLE_NAME)){
386386
description = "The hbase:meta table holds references to all User Table regions.";
387-
} else if (tableName.equals(Canary.DEFAULT_WRITE_TABLE_NAME)){
387+
} else if (tableName.equals(CanaryTool.DEFAULT_WRITE_TABLE_NAME)){
388388
description = "The hbase:canary table is used to sniff the write availbility of"
389389
+ " each regionserver.";
390390
} else if (tableName.equals(AccessControlLists.ACL_TABLE_NAME)){

0 commit comments

Comments
 (0)