Skip to content

Commit 52dc34c

Browse files
bsglzGuanghao Zhang
authored andcommitted
HBASE-23334 The table-lock node of zk is not needed since HBASE-16786 (apache#873)
Signed-off-by: Guangxu Cheng <gxcheng@apache.org>
1 parent 4ace94d commit 52dc34c

3 files changed

Lines changed: 0 additions & 6 deletions

File tree

hbase-client/src/main/java/org/apache/hadoop/hbase/zookeeper/ZNodePaths.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,8 +75,6 @@ public class ZNodePaths {
7575
public final String regionNormalizerZNode;
7676
// znode containing the state of all switches, currently there are split and merge child node.
7777
public final String switchZNode;
78-
// znode containing the lock for the tables
79-
public final String tableLockZNode;
8078
// znode containing namespace descriptors
8179
public final String namespaceZNode;
8280
// znode of indicating master maintenance mode
@@ -116,7 +114,6 @@ public ZNodePaths(Configuration conf) {
116114
regionNormalizerZNode =
117115
joinZNode(baseZNode, conf.get("zookeeper.znode.regionNormalizer", "normalizer"));
118116
switchZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.switch", "switch"));
119-
tableLockZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.tableLock", "table-lock"));
120117
namespaceZNode = joinZNode(baseZNode, conf.get("zookeeper.znode.namespace", "namespace"));
121118
masterMaintZNode =
122119
joinZNode(baseZNode, conf.get("zookeeper.znode.masterMaintenance", "master-maintenance"));
@@ -146,7 +143,6 @@ public String toString() {
146143
.append(", balancerZNode=").append(balancerZNode)
147144
.append(", regionNormalizerZNode=").append(regionNormalizerZNode)
148145
.append(", switchZNode=").append(switchZNode)
149-
.append(", tableLockZNode=").append(tableLockZNode)
150146
.append(", namespaceZNode=").append(namespaceZNode)
151147
.append(", masterMaintZNode=").append(masterMaintZNode)
152148
.append(", replicationZNode=").append(replicationZNode)

hbase-client/src/test/java/org/apache/hadoop/hbase/zookeeper/TestZNodePaths.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ public void testIsClientReadable() {
4646
assertTrue(znodePaths.isClientReadable(ZNodePaths.joinZNode(znodePaths.tableZNode, "foo")));
4747
assertTrue(znodePaths.isClientReadable(znodePaths.rsZNode));
4848

49-
assertFalse(znodePaths.isClientReadable(znodePaths.tableLockZNode));
5049
assertFalse(znodePaths.isClientReadable(znodePaths.balancerZNode));
5150
assertFalse(znodePaths.isClientReadable(znodePaths.regionNormalizerZNode));
5251
assertFalse(znodePaths.isClientReadable(znodePaths.clusterStateZNode));

hbase-zookeeper/src/main/java/org/apache/hadoop/hbase/zookeeper/ZKWatcher.java

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -192,7 +192,6 @@ private void createBaseZNodes() throws ZooKeeperConnectionException {
192192
ZKUtil.createAndFailSilent(this, znodePaths.tableZNode);
193193
ZKUtil.createAndFailSilent(this, znodePaths.splitLogZNode);
194194
ZKUtil.createAndFailSilent(this, znodePaths.backupMasterAddressesZNode);
195-
ZKUtil.createAndFailSilent(this, znodePaths.tableLockZNode);
196195
ZKUtil.createAndFailSilent(this, znodePaths.masterMaintZNode);
197196
} catch (KeeperException e) {
198197
throw new ZooKeeperConnectionException(

0 commit comments

Comments
 (0)