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] InternalGeoCentroidTests#testReduceRandom fails on 6.x #29480

Closed
dnhatn opened this issue Apr 11, 2018 · 1 comment
Closed

[CI] InternalGeoCentroidTests#testReduceRandom fails on 6.x #29480

dnhatn opened this issue Apr 11, 2018 · 1 comment
Assignees
@imotov
Labels
:Analytics/Aggregations Aggregations >test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI

Comments

@dnhatn
Copy link
Member

dnhatn commented Apr 11, 2018

InternalGeoCentroidTests#testReduceRandom fails on 6.x branch.

InternalGeoCentroidTests.testReduceRandom <<< FAILURES!
   > Throwable #1: java.lang.NullPointerException
   >    at __randomizedtesting.SeedInfo.seed([6447FA9D630D8B60:A47A568C45501236]:0)
   >    at org.elasticsearch.search.aggregations.metrics.geocentroid.InternalGeoCentroidTests.assertReduced(InternalGeoCentroidTests.java:71)
   >    at org.elasticsearch.search.aggregations.metrics.geocentroid.InternalGeoCentroidTests.assertReduced(InternalGeoCentroidTests.java:34)
   >    at org.elasticsearch.test.InternalAggregationTestCase.testReduceRandom(InternalAggregationTestCase.java:262)
   >    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
   >    at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
   >    at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
   >    at java.base/java.lang.reflect.Method.invoke(Method.java:564)
   >    at java.base/java.lang.Thread.run(Thread.java:844)

I can reproduce this locally on 6.x branch.

./gradlew :server:test -Dtests.seed=6447FA9D630D8B60 -Dtests.class=org.elasticsearch.search.aggregations.metrics.geocentroid.InternalGeoCentroidTests -Dtests.method="testReduceRandom" -Dtests.security.manager=true -Dtests.locale=sv -Dtests.timezone=America/Eirunepe

CI: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+ccr-6.x+feature-branch/50/console
@dnhatn dnhatn added >test Issues or PRs that are addressing/adding tests :Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI labels Apr 11, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-search-aggs

imotov added a commit to imotov/elasticsearch that referenced this issue Apr 11, 2018
@imotov
Fix NPE in InternalGeoCentroidTests#testReduceRandom
9cc7866 
In some rare cases all inputs might have 0 count and resulting in
zero totalCount, and null in centroid causing NPE.

Closes elastic#29480
@imotov imotov mentioned this issue Apr 11, 2018
Merged
@imotov imotov self-assigned this Apr 11, 2018
imotov added a commit that referenced this issue Apr 12, 2018
@imotov
Fix NPE in InternalGeoCentroidTests#testReduceRandom (#29481)
0aa1918 
In some rare cases all inputs might have zero count and resulting in
zero totalCount, and null in centroid causing NPE.

Closes #29480
imotov added a commit that referenced this issue Apr 12, 2018
@imotov
Fix NPE in InternalGeoCentroidTests#testReduceRandom (#29481)
029bfcd 
In some rare cases all inputs might have zero count and resulting in
zero totalCount, and null in centroid causing NPE.

Closes #29480
imotov added a commit that referenced this issue Apr 12, 2018
@imotov
Fix NPE in InternalGeoCentroidTests#testReduceRandom (#29481)
ccec39f 
In some rare cases all inputs might have zero count and resulting in
zero totalCount, and null in centroid causing NPE.

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

@imotov imotov

Labels
:Analytics/Aggregations Aggregations >test Issues or PRs that are addressing/adding tests >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

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