Skip to content

Commit e153f58

Browse files
virajjasanipetersomogyi
authored andcommitted
HBASE-22633 remove redundant substring call with append for ZKReplicationQueueStorage
Signed-off-by: Sakthi <sakthivel.azhaku@gmail.com> Signed-off-by: Duo Zhang <zhangduo@apache.org>
1 parent 30ccac5 commit e153f58

1 file changed

Lines changed: 4 additions & 3 deletions

File tree

hbase-replication/src/main/java/org/apache/hadoop/hbase/replication/ZKReplicationQueueStorage.java

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -165,9 +165,10 @@ String getSerialReplicationRegionPeerNode(String encodedRegionName, String peerI
165165
"Invalid encoded region name: " + encodedRegionName + ", length should be 32.");
166166
}
167167
return new StringBuilder(regionsZNode).append(ZNodePaths.ZNODE_PATH_SEPARATOR)
168-
.append(encodedRegionName.substring(0, 2)).append(ZNodePaths.ZNODE_PATH_SEPARATOR)
169-
.append(encodedRegionName.substring(2, 4)).append(ZNodePaths.ZNODE_PATH_SEPARATOR)
170-
.append(encodedRegionName.substring(4)).append("-").append(peerId).toString();
168+
.append(encodedRegionName, 0, 2).append(ZNodePaths.ZNODE_PATH_SEPARATOR)
169+
.append(encodedRegionName, 2, 4).append(ZNodePaths.ZNODE_PATH_SEPARATOR)
170+
.append(encodedRegionName, 4, encodedRegionName.length()).append("-").append(peerId)
171+
.toString();
171172
}
172173

173174
@Override

0 commit comments

Comments
 (0)