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.

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

[CI] test failure in o.o.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket #1998

Closed
nknize opened this issue Jan 27, 2022 · 4 comments
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run Search:Aggregations >test-failure Test failure from CI, local build, etc. v2.0.0 Version 2.0.0

Comments

@nknize
Copy link
Collaborator

nknize commented Jan 27, 2022

Reproducible test failure related to #1942

REPRODUCE WITH: ./gradlew ':server:test' --tests "org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket" -Dtests.seed=7AEBC6A11333343F -Dtests.security.manager=true -Dtests.jvm.argline="-XX:TieredStopAtLevel=1 -XX:ReservedCodeCacheSize=64m" -Dtests.locale=es-PY -Dtests.timezone=Pacific/Funafuti -Druntime.java=17
org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests > testWithKeywordLongAndMissingBucket FAILED
    java.lang.AssertionError: expected:<3> but was:<4>
        at __randomizedtesting.SeedInfo.seed([7AEBC6A11333343F:9BD426C674ED5A70]:0)
        at org.junit.Assert.fail(Assert.java:89)
        at org.junit.Assert.failNotEquals(Assert.java:835)
        at org.junit.Assert.assertEquals(Assert.java:647)
        at org.junit.Assert.assertEquals(Assert.java:633)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.lambda$testWithKeywordLongAndMissingBucket$93(CompositeAggregatorTests.java:1139)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.executeTestCase(CompositeAggregatorTests.java:2712)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testSearchCase(CompositeAggregatorTests.java:2660)
        at org.opensearch.search.aggregations.bucket.composite.CompositeAggregatorTests.testWithKeywordLongAndMissingBucket(CompositeAggregatorTests.java:1128)
@nknize nknize added bug Something isn't working >test-failure Test failure from CI, local build, etc. v2.0.0 Version 2.0.0 untriaged Search:Aggregations labels Jan 27, 2022
@nknize
Copy link
Collaborator Author

nknize commented Jan 27, 2022

@penghuo can you have a look at this test failure?

@penghuo
Copy link
Contributor

penghuo commented Jan 27, 2022

@penghuo can you have a look at this test failure?

Ack

@penghuo
Copy link
Contributor

penghuo commented Jan 27, 2022

The root cause of the issue is InternalBucket does not use MissingOrders. In bucket reduce phase, null bucket's order is less than others.
The test cases not always fail because the bug in bucket reduce phase which required at least 2 shards.
I am preparing PR now, will publish soon.

@anasalkouz anasalkouz added flaky-test Random test failure that succeeds on second run and removed untriaged labels Feb 1, 2022
@penghuo
Copy link
Contributor

penghuo commented Feb 18, 2022

@nknize Could we close this issue? Failed test already been fixed by #2005.

@dblock dblock closed this as completed Feb 19, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working flaky-test Random test failure that succeeds on second run Search:Aggregations >test-failure Test failure from CI, local build, etc. v2.0.0 Version 2.0.0
Projects
None yet
Development

No branches or pull requests

4 participants