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

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


The following commit(s) were added to refs/heads/main by this push:
     new d26c786abe Fixed MemoryStarvedScanIT (#5106)
d26c786abe is described below

commit d26c786abe387f9c07c0861563258a384c5e2c1b
Author: Dave Marion <dlmar...@apache.org>
AuthorDate: Mon Nov 25 10:34:13 2024 -0500

    Fixed MemoryStarvedScanIT (#5106)
    
    Test started failing after recent metrics changes. The comment
    above the code block mentioned repeating the same check, but the
    code wasn't doing that. Modifying the code to do the same check
    fixed the issue.
---
 .../java/org/apache/accumulo/test/functional/MemoryStarvedScanIT.java   | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git 
a/test/src/main/java/org/apache/accumulo/test/functional/MemoryStarvedScanIT.java
 
b/test/src/main/java/org/apache/accumulo/test/functional/MemoryStarvedScanIT.java
index 1320a6018f..78edc22525 100644
--- 
a/test/src/main/java/org/apache/accumulo/test/functional/MemoryStarvedScanIT.java
+++ 
b/test/src/main/java/org/apache/accumulo/test/functional/MemoryStarvedScanIT.java
@@ -547,7 +547,7 @@ public class MemoryStarvedScanIT extends 
SharedMiniClusterBase {
         Thread.sleep(1500);
         assertEquals(currentCount, fetched.get());
         assertTrue(SCAN_START_DELAYED.doubleValue() >= paused);
-        assertEquals(returned, SCAN_RETURNED_EARLY.doubleValue());
+        assertTrue(SCAN_RETURNED_EARLY.doubleValue() >= returned);
 
         // check across multiple low memory checks and metric updates that low 
memory detected
         // remains set

Reply via email to