Skip to content

Commit

Permalink
ESQL: Reenable heap attacks (elastic#121565)
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 elastic#121481 Close elastic#121465
  • Loading branch information
nik9000 committed Feb 3, 2025
1 parent 86de107 commit dcf9deb
Showing 1 changed file with 4 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -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);
});
}
Expand Down Expand Up @@ -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);
});
}
Expand Down

0 comments on commit dcf9deb

Please sign in to comment.