Skip to content

Commit a8b9396

Browse files
committed
ESQL: Reenable heap attacks (elastic#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 elastic#121481 Close elastic#121465
1 parent 5ed0397 commit a8b9396

File tree

1 file changed

+4
-2
lines changed
  • test/external-modules/esql-heap-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack

1 file changed

+4
-2
lines changed

test/external-modules/esql-heap-attack/src/javaRestTest/java/org/elasticsearch/xpack/esql/heap_attack/HeapAttackIT.java

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -636,7 +636,8 @@ public void testLookupExplosion() throws IOException {
636636

637637
public void testLookupExplosionManyMatches() throws IOException {
638638
assertCircuitBreaks(() -> {
639-
Map<?, ?> result = lookupExplosion(1500, 10000);
639+
// 1500, 10000 is enough locally, but some CI machines need more.
640+
Map<?, ?> result = lookupExplosion(2000, 10000);
640641
logger.error("should have failed but got {}", result);
641642
});
642643
}
@@ -664,7 +665,8 @@ public void testLookupExplosionBigString() throws IOException {
664665

665666
public void testLookupExplosionBigStringManyMatches() throws IOException {
666667
assertCircuitBreaks(() -> {
667-
Map<?, ?> result = lookupExplosionBigString(500, 1);
668+
// 500, 1 is enough to make it fail locally but some CI needs more
669+
Map<?, ?> result = lookupExplosionBigString(800, 1);
668670
logger.error("should have failed but got {}", result);
669671
});
670672
}

0 commit comments

Comments
 (0)