ddanielr commented on code in PR #3738:
URL: https://github.com/apache/accumulo/pull/3738#discussion_r1333859978
##########
server/gc/src/test/java/org/apache/accumulo/gc/GarbageCollectionTest.java:
##########
@@ -863,6 +915,45 @@ public void testMissingTableIds() throws Exception {
&& msg.contains("Saw table IDs in ZK that were not in metadata
table:"), msg);
}
+ @Test
+ public void testDeletingInUseReferenceCandidates() throws Exception {
Review Comment:
Added the GcCandidateType to the standard `assertCandidateRemoved` method
and ensured the proper type was passed for each test.
I also added the `assertNoCandidatesRemoved(TestGCE gce)` method so that a
unused GcCandidateType did not need to be added for validating an empty
candidates TreeMap
--
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]
