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

test failure: org.elasticsearch.xpack.analytics.normalize.NormalizeAggregatorTests.testTermsAggParent #57164

Closed
jakelandis opened this issue May 26, 2020 · 2 comments · Fixed by #57171
Closed

test failure: org.elasticsearch.xpack.analytics.normalize.NormalizeAggregatorTests.testTermsAggParent #57164

jakelandis opened this issue May 26, 2020 · 2 comments · Fixed by #57171
Labels
:Analytics/Aggregations Aggregations Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI v7.9.0 v8.0.0-alpha1

Comments

@jakelandis
Copy link
Contributor

Build scan:

https://gradle-enterprise.elastic.co/s/zb66vt4jdhzgk

Repro line:

./gradlew ':x-pack:plugin:analytics:test' --tests "org.elasticsearch.xpack.analytics.normalize.NormalizeAggregatorTests.testTermsAggParent" -Dtests.seed=921D46F82701E024 -Dtests.security.manager=true -Dtests.locale=sq -Dtests.timezone=America/Yellowknife -Dcompiler.java=14 -Druntime.java=11

Reproduces locally?:

Yes.

Applicable branches:

7.x

Failure history:

Recently added feature: #56399 cc: @talevy

Failure excerpt:

org.elasticsearch.xpack.analytics.normalize.NormalizeAggregatorTests > testTermsAggParent FAILED
    java.lang.ClassCastException: class org.apache.lucene.index.ParallelCompositeReader cannot be cast to class org.apache.lucene.index.DirectoryReader (org.apache.lucene.index.ParallelCompositeReader and org.apache.lucene.index.DirectoryReader are in unnamed module of loader 'app')
        at __randomizedtesting.SeedInfo.seed([921D46F82701E024:928D3EDD3E4FD870]:0)
        at org.elasticsearch.search.aggregations.support.ValuesSource$Bytes$WithOrdinals$FieldData.globalOrdinalsValues(ValuesSource.java:206)
        at org.elasticsearch.search.aggregations.support.ValuesSource$Bytes$WithOrdinals.globalMaxOrd(ValuesSource.java:179)
        at org.elasticsearch.search.aggregations.bucket.terms.TermsAggregatorFactory.getMaxOrd(TermsAggregatorFactory.java:300)
        at org.elasticsearch.search.aggregations.bucket.terms.TermsAggregatorFactory.access$100(TermsAggregatorFactory.java:52)
        at org.elasticsearch.search.aggregations.bucket.terms.TermsAggregatorFactory$1.build(TermsAggregatorFactory.java:108)
        at org.elasticsearch.search.aggregations.bucket.terms.TermsAggregatorFactory.doCreateInternal(TermsAggregatorFactory.java:274)
        at org.elasticsearch.search.aggregations.support.ValuesSourceAggregatorFactory.createInternal(ValuesSourceAggregatorFactory.java:49)
        at org.elasticsearch.search.aggregations.AggregatorFactory.create(AggregatorFactory.java:250)
        at org.elasticsearch.search.aggregations.AggregatorTestCase.createAggregator(AggregatorTestCase.java:244)
        at org.elasticsearch.search.aggregations.AggregatorTestCase.createAggregator(AggregatorTestCase.java:222)
        at org.elasticsearch.search.aggregations.AggregatorTestCase.searchAndReduce(AggregatorTestCase.java:493)
        at org.elasticsearch.search.aggregations.AggregatorTestCase.searchAndReduce(AggregatorTestCase.java:441)
        at org.elasticsearch.xpack.analytics.normalize.NormalizeAggregatorTests.testCase(NormalizeAggregatorTests.java:165)
        at org.elasticsearch.xpack.analytics.normalize.NormalizeAggregatorTests.testTermsAggParent(NormalizeAggregatorTests.java:120)
@jakelandis jakelandis added :Analytics/Aggregations Aggregations >test-failure Triaged test failures from CI labels May 26, 2020
@elasticmachine
Copy link
Collaborator

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

@elasticmachine elasticmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label May 26, 2020
@talevy
Copy link
Contributor

talevy commented May 26, 2020

what in the world. I'll investigate. thanks @jakelandis

@talevy talevy self-assigned this May 26, 2020
talevy added a commit to talevy/elasticsearch that referenced this issue May 26, 2020
@talevy
Fix NormalizerAgg test searcher wrapping
9f63841 
The searcher was randomly wrapping its reader as slow, parallel, or filtered.
This was causing casting issues in the normalizer tests. By removing the
wrapping, the problem goes away.

Closes elastic#57164
@talevy talevy mentioned this issue May 26, 2020
Merged
talevy added a commit that referenced this issue May 26, 2020
@talevy
Fix NormalizerAgg test searcher wrapping (#57171)
7eb5273 
The searcher was randomly wrapping its reader as slow, parallel, or filtered.
This was causing casting issues in the normalizer tests. By removing the
wrapping, the problem goes away.

Closes #57164
talevy added a commit that referenced this issue May 26, 2020
@talevy
Fix NormalizerAgg test searcher wrapping (#57171)
8106082 
The searcher was randomly wrapping its reader as slow, parallel, or filtered.
This was causing casting issues in the normalizer tests. By removing the
wrapping, the problem goes away.

Closes #57164
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Analytics/Aggregations Aggregations Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI v7.9.0 v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix NormalizerAgg test searcher wrapping talevy/elasticsearch
3 participants
@talevy @jakelandis @elasticmachine