Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI] CompositeAggregatorTests testRandomLongs failing #78919

Closed
martijnvg opened this issue Oct 11, 2021 · 2 comments · Fixed by #78926
Closed

[CI] CompositeAggregatorTests testRandomLongs failing #78919

martijnvg opened this issue Oct 11, 2021 · 2 comments · Fixed by #78926
Assignees
@jimczi
Labels
Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI

Comments

@martijnvg
Copy link
Member

Build scan:
https://gradle-enterprise.elastic.co/s/uqiesaacvzq7g/tests/:server:test/org.elasticsearch.search.aggregations.bucket.composite.CompositeAggregatorTests/testRandomLongs

Reproduction line:
./gradlew ':server:test' --tests "org.elasticsearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testRandomLongs" -Dtests.seed=DB61B511F6F236FA -Dtests.locale=de-CH -Dtests.timezone=Antarctica/Casey -Druntime.java=11

Applicable branches:
master

Reproduces locally?:
Yes

Failure history:
https://gradle-enterprise.elastic.co/scans/tests?tests.container=org.elasticsearch.search.aggregations.bucket.composite.CompositeAggregatorTests&tests.test=testRandomLongs

Failure excerpt:

java.lang.OutOfMemoryError: Java heap space
@martijnvg martijnvg added :Analytics/CompositeAggs >test-failure Triaged test failures from CI labels Oct 11, 2021
@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 11, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-analytics-geo (Team:Analytics)

@nik9000 nik9000 self-assigned this Oct 11, 2021
@jimczi jimczi assigned jimczi and unassigned nik9000 Oct 11, 2021
@jimczi jimczi mentioned this issue Oct 11, 2021
Merged
@ywangd
Copy link
Member

ywangd commented Oct 11, 2021

This failure for CompositeAggregatorTests.testRandomStrings is likely due to the same underlying issue. Error message is:

java.lang.OutOfMemoryError: Java heap space

Build scan: https://gradle-enterprise.elastic.co/s/y2zillkv3lvw6/tests/:server:test/org.elasticsearch.search.aggregations.bucket.composite.CompositeAggregatorTests/testRandomStrings?top-execution=1

Reproducible locally with :

./gradlew ':server:test' --tests "org.elasticsearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testRandomStrings" -Dtests.seed=8E4872DF11193E13 -Dtests.locale=it -Dtests.timezone=Asia/Qatar -Druntime.java=11

jimczi added a commit that referenced this issue Oct 12, 2021
@jimczi
Fix failures in CompositeAggregatorTests
68bc320 
The random tests in CompositeAggregatorTests create lots of segments since #78313. That can lead to out of memory in tests.
The additional commits were added to simulate the multi-segments case but the random index writer should already perform some random overcommits.

Closes #78919
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@jimczi jimczi

Labels
Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix failures in CompositeAggregatorTests jimczi/elasticsearch
5 participants
@nik9000 @martijnvg @ywangd @elasticmachine @jimczi