EdColeman commented on code in PR #4201:
URL: https://github.com/apache/accumulo/pull/4201#discussion_r1470322250
##########
test/src/main/java/org/apache/accumulo/test/functional/CompactionIT.java:
##########
@@ -474,10 +474,12 @@ public void testConcurrentSplit() throws Exception {
// wait a bit for some tablets to have files selected, it possible the
compaction have
// completed before this so do not wait long
- Wait.waitFor(
- () -> countTablets(tableName, tabletMetadata ->
tabletMetadata.getSelectedFiles() != null)
- > 0,
- 3000, 10);
+ long startTime = System.nanoTime();
Review Comment:
A comment here explaining why waitFor will not work could help prevent
someone from seeing this and thinking that it could be converted back. (applies
in two places)
--
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]
