zhengzhuobinzzb commented on a change in pull request #3876:
URL: https://github.com/apache/hbase/pull/3876#discussion_r755785944
##########
File path:
hbase-replication/src/test/java/org/apache/hadoop/hbase/replication/TestZKReplicationQueueStorage.java
##########
@@ -206,18 +206,22 @@ public void testAddRemoveLog() throws
ReplicationException {
}
}
- // For HBASE-12865
+ // For HBASE-12865, HBASE-26482
Review comment:
Method claimQueue will not remove server znode. And we don't need to
delete the server znode to test the update of cversion. We just need to test
claimQueue will update cversion of root queuesZnode.
So i reuse this method for cversion update test. But add queue '2' to
enhance the robustness of the test code (to prevent cversion from changing
because the server znode is deleted)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
