Skip to content

Commit 61f0d1d

Browse files
committed
Check that all compactions are system compactions in the wait condition
Noticed that one of the tests failed the assertion, causing the test to fail instead of retrying the wait loop.
1 parent 1fcf88d commit 61f0d1d

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/src/main/java/org/apache/accumulo/test/functional/CompactionIT.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -997,9 +997,9 @@ private void testCancelUserCompactionTimeout(boolean timeout) throws Exception {
997997
Wait.waitFor(() -> {
998998
var tabletMeta = ((ClientContext) client).getAmple().readTablet(extent);
999999
var externalCompactions = tabletMeta.getExternalCompactions();
1000-
assertTrue(externalCompactions.values().stream()
1001-
.allMatch(ec -> ec.getKind() == CompactionKind.SYSTEM));
1002-
return externalCompactions.size() == 1;
1000+
boolean allECAreSystem = externalCompactions.values().stream()
1001+
.allMatch(ec -> ec.getKind() == CompactionKind.SYSTEM);
1002+
return allECAreSystem && externalCompactions.size() == 1;
10031003
}, Wait.MAX_WAIT_MILLIS, 10);
10041004

10051005
// Wait for the user compaction to now run after the system finishes

0 commit comments

Comments
 (0)