(accumulo) branch 2.1 updated: ExternalCompactionProgressIT - verify only after compaction completes (#4661)

[domgarguilo]

This is an automated email from the ASF dual-hosted git repository.

domgarguilo pushed a commit to branch 2.1
in repository https://gitbox.apache.org/repos/asf/accumulo.git


The following commit(s) were added to refs/heads/2.1 by this push:
     new 9ce9ff9d74 ExternalCompactionProgressIT - verify only after compaction 
completes (#4661)
9ce9ff9d74 is described below

commit 9ce9ff9d74e06e8603f51b14f4c22fef710d8259
Author: Dom G <domgargu...@apache.org>
AuthorDate: Tue Jun 11 08:38:42 2024 -0400

    ExternalCompactionProgressIT - verify only after compaction completes 
(#4661)
---
 .../accumulo/test/compaction/ExternalCompactionProgressIT.java      | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git 
a/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompactionProgressIT.java
 
b/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompactionProgressIT.java
index c57705807a..f3f8864bb3 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompactionProgressIT.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/compaction/ExternalCompactionProgressIT.java
@@ -169,11 +169,11 @@ public class ExternalCompactionProgressIT extends 
AccumuloClusterHarness {
       }, 30_000, CHECKER_THREAD_SLEEP_MS,
           "Entries read and written metrics values did not match expected 
values");
 
-      log.info("Done Compacting table");
-      verify(client, table, 2, ROWS);
-
       Wait.waitFor(() -> compactorBusy.get() == 0, 30_000, 
CHECKER_THREAD_SLEEP_MS,
           "Compactor busy metric should be false once compaction completes");
+
+      log.info("Done Compacting table");
+      verify(client, table, 2, ROWS);
     } finally {
       stopCheckerThread.set(true);
       checkerThread.join();