diff --git a/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZNodePaths.java b/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZNodePaths.java index b9a34120fdd8..87cc94d7cdf4 100644 --- a/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZNodePaths.java +++ b/hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZNodePaths.java @@ -74,8 +74,6 @@ public class ZNodePaths { public final String regionNormalizerZNode; // znode containing the state of all switches, currently there are split and merge child node. public final String switchZNode; - // znode containing the lock for the tables - public final String tableLockZNode; // znode containing namespace descriptors public final String namespaceZNode; // znode of indicating master maintenance mode @@ -113,7 +111,6 @@ public ZNodePaths(Configuration conf) { regionNormalizerZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.regionNormalizer", "normalizer")); switchZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.switch", "switch")); - tableLockZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.tableLock", "table-lock")); namespaceZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.namespace", "namespace")); masterMaintZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.masterMaintenance", "master-maintenance")); @@ -127,16 +124,27 @@ public ZNodePaths(Configuration conf) { @Override public String toString() { - return "ZNodePaths [baseZNode=" + baseZNode + ", metaReplicaZNodes=" + metaReplicaZNodes - + ", rsZNode=" + rsZNode + ", drainingZNode=" + drainingZNode + ", masterAddressZNode=" - + masterAddressZNode + ", backupMasterAddressesZNode=" + backupMasterAddressesZNode - + ", clusterStateZNode=" + clusterStateZNode + ", tableZNode=" + tableZNode - + ", clusterIdZNode=" + clusterIdZNode + ", splitLogZNode=" + splitLogZNode - + ", balancerZNode=" + balancerZNode + ", regionNormalizerZNode=" + regionNormalizerZNode - + ", switchZNode=" + switchZNode + ", tableLockZNode=" + tableLockZNode - + ", namespaceZNode=" + namespaceZNode + ", masterMaintZNode=" + masterMaintZNode - + ", replicationZNode=" + replicationZNode + ", peersZNode=" + peersZNode - + ", queuesZNode=" + queuesZNode + ", hfileRefsZNode=" + hfileRefsZNode + "]"; + return new StringBuilder() + .append("ZNodePaths [baseZNode=").append(baseZNode) + .append(", metaReplicaZNodes=").append(metaReplicaZNodes) + .append(", rsZNode=").append(rsZNode) + .append(", drainingZNode=").append(drainingZNode) + .append(", masterAddressZNode=").append(masterAddressZNode) + .append(", backupMasterAddressesZNode=").append(backupMasterAddressesZNode) + .append(", clusterStateZNode=").append(clusterStateZNode) + .append(", tableZNode=").append(tableZNode) + .append(", clusterIdZNode=").append(clusterIdZNode) + .append(", splitLogZNode=").append(splitLogZNode) + .append(", balancerZNode=").append(balancerZNode) + .append(", regionNormalizerZNode=").append(regionNormalizerZNode) + .append(", switchZNode=").append(switchZNode) + .append(", namespaceZNode=").append(namespaceZNode) + .append(", masterMaintZNode=").append(masterMaintZNode) + .append(", replicationZNode=").append(replicationZNode) + .append(", peersZNode=").append(peersZNode) + .append(", queuesZNode=").append(queuesZNode) + .append(", hfileRefsZNode=").append(hfileRefsZNode) + .append("]").toString(); } /** diff --git a/hbase-client/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZNodePaths.java b/hbase-client/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZNodePaths.java index 1a45379edf6b..a8b7644c52af 100644 --- a/hbase-client/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZNodePaths.java +++ b/hbase-client/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZNodePaths.java @@ -46,7 +46,6 @@ public void testIsClientReadable() { assertTrue(znodePaths.isClientReadable(ZNodePaths.joinZNode(znodePaths.tableZNode, "foo"))); assertTrue(znodePaths.isClientReadable(znodePaths.rsZNode)); - assertFalse(znodePaths.isClientReadable(znodePaths.tableLockZNode)); assertFalse(znodePaths.isClientReadable(znodePaths.balancerZNode)); assertFalse(znodePaths.isClientReadable(znodePaths.regionNormalizerZNode)); assertFalse(znodePaths.isClientReadable(znodePaths.clusterStateZNode)); diff --git a/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java b/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java index ce00af40495f..e5ef089a87c4 100644 --- a/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java +++ b/hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java @@ -186,7 +186,6 @@ private void createBaseZNodes() throws ZooKeeperConnectionException { ZKUtil.createAndFailSilent(this, znodePaths.tableZNode); ZKUtil.createAndFailSilent(this, znodePaths.splitLogZNode); ZKUtil.createAndFailSilent(this, znodePaths.backupMasterAddressesZNode); - ZKUtil.createAndFailSilent(this, znodePaths.tableLockZNode); ZKUtil.createAndFailSilent(this, znodePaths.masterMaintZNode); } catch (KeeperException e) { throw new ZooKeeperConnectionException(