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: ScriptedMetricAggregatorTests testSelfReferencingAggStateAfterMap fails reproducibly #31307

Closed
imotov opened this issue Jun 13, 2018 · 2 comments
Closed

CI: ScriptedMetricAggregatorTests testSelfReferencingAggStateAfterMap fails reproducibly #31307

imotov opened this issue Jun 13, 2018 · 2 comments
Assignees
@cbuescher
Labels
:Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI

Comments

@imotov
Copy link
Contributor

imotov commented Jun 13, 2018

Found failure in PR build, but was able to reproduce it on master https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+pull-request/11646/console

  2> REPRODUCE WITH: ./gradlew :server:test -Dtests.seed=FB4E4946C6EABFE1 -Dtests.class=org.elasticsearch.search.aggregations.metrics.scripted.ScriptedMetricAggregatorTests -Dtests.method="testSelfReferencingAggStateAfterMap" -Dtests.security.manager=true -Dtests.locale=und -Dtests.timezone=Iceland
FAILURE 0.01s | ScriptedMetricAggregatorTests.testSelfReferencingAggStateAfterMap <<< FAILURES!
   > Throwable #1: junit.framework.AssertionFailedError: Expected exception IllegalArgumentException but no exception was thrown
   >    at __randomizedtesting.SeedInfo.seed([FB4E4946C6EABFE1:5C2FC010C227E5E2]:0)
   >    at org.apache.lucene.util.LuceneTestCase.expectThrows(LuceneTestCase.java:2686)
   >    at org.elasticsearch.search.aggregations.metrics.scripted.ScriptedMetricAggregatorTests.testSelfReferencingAggStateAfterMap(ScriptedMetricAggregatorTests.java:315)
@imotov imotov added :Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI labels Jun 13, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

imotov added a commit that referenced this issue Jun 13, 2018
@imotov
Mute ScriptedMetricAggregatorTests testSelfReferencingAggStateAfterMap
018d3fc 
Tracked by #31307
@cbuescher cbuescher self-assigned this Jun 15, 2018
@cbuescher
Copy link
Member

I think I found the problem, will open a PR.

@cbuescher cbuescher mentioned this issue Jun 15, 2018
Merged
cbuescher pushed a commit to cbuescher/elasticsearch that referenced this issue Jun 15, 2018
[Tests] Fix edge case in ScriptedMetricAggregatorTests
35fe5cf 
An expected exception is only thrown when there are documents in the index
created in the test setup. Fixed the test by making sure there is at least one.

Closes elastic#31307
cbuescher pushed a commit that referenced this issue Jun 15, 2018
[Tests] Fix edge case in ScriptedMetricAggregatorTests (#31357)
fec7860 
An expected exception is only thrown when there are documents in the index
created in the test setup. Fixed the test by making sure there is at least one.

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

@cbuescher cbuescher

Labels
:Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@imotov @cbuescher @elasticmachine