From dcf9debae14ea6f42c1df4fbd8f425b8e689393b Mon Sep 17 00:00:00 2001 From: Nik Everett Date: Mon, 3 Feb 2025 16:02:30 -0500 Subject: [PATCH] ESQL: Reenable heap attacks (#121565) Reenables some heap attack tests, bumping their memory requirements to try and force a failure on all CI machines. Previously some CI machines weren't failing, invalidating the test on those machines. Close #121481 Close #121465 --- .../elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/test/external-modules/esql-heap-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java b/test/external-modules/esql-heap-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java index 59ce3c2dbb1cf..aaea04997bf3b 100644 --- a/test/external-modules/esql-heap-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java +++ b/test/external-modules/esql-heap-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java @@ -636,7 +636,8 @@ public void testLookupExplosion() throws IOException { public void testLookupExplosionManyMatches() throws IOException { assertCircuitBreaks(() -> { - Map result = lookupExplosion(1500, 10000); + // 1500, 10000 is enough locally, but some CI machines need more. + Map result = lookupExplosion(2000, 10000); logger.error("should have failed but got {}", result); }); } @@ -664,7 +665,8 @@ public void testLookupExplosionBigString() throws IOException { public void testLookupExplosionBigStringManyMatches() throws IOException { assertCircuitBreaks(() -> { - Map result = lookupExplosionBigString(500, 1); + // 500, 1 is enough to make it fail locally but some CI needs more + Map result = lookupExplosionBigString(800, 1); logger.error("should have failed but got {}", result); }); }