Skip to content

Commit 8f79947

Browse files
authored
ESQL: Reenable heap attacks (#121565) (#121573)
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
1 parent 9c4e02b commit 8f79947

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
@@ -635,7 +635,8 @@ public void testLookupExplosion() throws IOException {
635635

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

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

0 commit comments

Comments
 (0)