Skip to content

Commit

Permalink
Unmute HeapAttack tests (#102942)
Browse files Browse the repository at this point in the history
This PR re-enables two more heap attack tests. I have run more than 
100 iterations with these tests without hitting any failures.
  • Loading branch information
dnhatn authored Dec 6, 2023
1 parent 1617a8d commit 5c3d118
Showing 1 changed file with 0 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -113,7 +113,6 @@ public void testGroupOnSomeLongs() throws IOException {
/**
* This groups on 5000 columns which used to throw a {@link StackOverflowError}.
*/
@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/100640")
public void testGroupOnManyLongs() throws IOException {
initManyLongs();
Map<?, ?> map = XContentHelper.convertToMap(
Expand Down Expand Up @@ -182,7 +181,6 @@ private Response concat(int evals) throws IOException {
/**
* Returns many moderately long strings.
*/
@AwaitsFix(bugUrl = "https://github.com/elastic/elasticsearch/issues/100678")
public void testManyConcat() throws IOException {
initManyLongs();
Map<?, ?> map = XContentHelper.convertToMap(JsonXContent.jsonXContent, EntityUtils.toString(manyConcat(300).getEntity()), false);
Expand Down

0 comments on commit 5c3d118

Please sign in to comment.