You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@accumulo.apache.org by do...@apache.org on 2025/10/31 17:55:10 UTC

(accumulo) branch 2.1 updated: Fix Flaky ClassLoaderContextCompactionIT (#5967)

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 c92c2246a7 Fix Flaky ClassLoaderContextCompactionIT (#5967)
c92c2246a7 is described below

commit c92c2246a7bb5d0c848175448e54e8573bc3a087
Author: Dom G. <do...@apache.org>
AuthorDate: Fri Oct 31 13:55:03 2025 -0400

    Fix Flaky ClassLoaderContextCompactionIT (#5967)
    
    * Fix Flakey ClassLoaderContextCompactionIT by correctly keeping track of incoming metrics values
---
 .../apache/accumulo/test/compaction/ClassLoaderContextCompactionIT.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/test/src/main/java/org/apache/accumulo/test/compaction/ClassLoaderContextCompactionIT.java b/test/src/main/java/org/apache/accumulo/test/compaction/ClassLoaderContextCompactionIT.java
index 15a539bf5c..f71e509945 100644
--- a/test/src/main/java/org/apache/accumulo/test/compaction/ClassLoaderContextCompactionIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/compaction/ClassLoaderContextCompactionIT.java
@@ -141,7 +141,7 @@ public class ClassLoaderContextCompactionIT extends AccumuloClusterHarness {
           } else if (s.startsWith(MetricsProducer.METRICS_COMPACTOR_FAILURES_CONSECUTIVE)) {
             Metric m = TestStatsDSink.parseStatsDMetric(s);
             LOG.info("{}", m);
-            consecutive.set(Long.parseLong(m.getValue()));
+            consecutive.getAndUpdate(prev -> Math.max(prev, Long.parseLong(m.getValue())));
           }
 
         }