Skip to content

Commit

Permalink
ESQL: Reenable heap attacks (#121565) (#121576)
Browse files Browse the repository at this point in the history
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
  • Loading branch information
nik9000 authored Feb 4, 2025
1 parent 2bc0d28 commit 10cf628
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -635,7 +635,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);
});
}
Expand Down Expand Up @@ -663,7 +664,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);
});
}
Expand Down

0 comments on commit 10cf628

Please sign in to comment.