DomGarguilo commented on code in PR #6073:
URL: https://github.com/apache/accumulo/pull/6073#discussion_r2728808196
##########
server/base/src/test/java/org/apache/accumulo/server/util/AdminTest.java:
##########
@@ -93,4 +102,114 @@ public void testCannotQualifySessionId() {
EasyMock.verify(zc);
}
+ /**
+ * SServer group filter should use lock data (UUID,group).
+ */
+ @SuppressWarnings("deprecation")
+ @Test
+ public void testSserverGroupFilterUsesLockData() throws Exception {
+
+ ZooReaderWriter zoo = EasyMock.createNiceMock(ZooReaderWriter.class);
+ ZooKeeper zk = EasyMock.createNiceMock(ZooKeeper.class);
+
+ String basePath = "/accumulo/iid/sservers";
+ String hostDefault = "host1:10000";
+ String hostOther = "host2:10001";
+ String zlock1 = "zlock#00000000-0000-0000-0000-aaaaaaaaaaaa#0000000001";
+ String zlock2 = "zlock#00000000-0000-0000-0000-bbbbbbbbbbbb#0000000001";
+
+ EasyMock.expect(zoo.exists(basePath)).andReturn(true);
+ EasyMock.expect(zoo.getChildren(basePath)).andReturn(List.of(hostDefault,
hostOther));
+ EasyMock.expect(zoo.getZooKeeper()).andReturn(zk);
+ EasyMock.expect(zk.getChildren(basePath + "/" + hostDefault,
null)).andReturn(List.of(zlock1));
+ EasyMock.expect(zk.getData(basePath + "/" + hostDefault + "/" + zlock1,
false, null))
+ .andReturn((UUID.randomUUID().toString() +
",default").getBytes(UTF_8));
+ EasyMock.expect(zk.getChildren(basePath + "/" + hostOther,
null)).andReturn(List.of(zlock2));
+ EasyMock.expect(zk.getData(basePath + "/" + hostOther + "/" + zlock2,
false, null))
+ .andReturn((UUID.randomUUID().toString() + ",rg1").getBytes(UTF_8));
+
+ AtomicBoolean deletedDefault = new AtomicBoolean(false);
Review Comment:
I was making a small PR against this branch for this test so included a
commit in that to do this too
https://github.com/dlmarion/accumulo/pull/58
--
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]
