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