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: IndexShardTests#testDocStats fails #32449

Closed
dnhatn opened this issue Jul 28, 2018 · 2 comments
Closed

CI: IndexShardTests#testDocStats fails #32449

dnhatn opened this issue Jul 28, 2018 · 2 comments
Labels
:Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >test-failure Triaged test failures from CI

Comments

@dnhatn
Copy link
Member

dnhatn commented Jul 28, 2018

This test started failing after we upgraded to Lucene-7.5.

I can reproduce this locally.

./gradlew :server:test \
  -Dtests.seed=B201D9968D50FB58 \
  -Dtests.class=org.elasticsearch.index.shard.IndexShardTests \
  -Dtests.method="testDocStats" \
  -Dtests.security.manager=true \
  -Dtests.locale=sr-CS \
  -Dtests.timezone=Asia/Makassar

CI: https://elasticsearch-ci.elastic.co/job/elastic+elasticsearch+master+multijob-windows-compatibility/1785/console
@dnhatn dnhatn added >test-failure Triaged test failures from CI :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. labels Jul 28, 2018
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed

dnhatn added a commit that referenced this issue Jul 28, 2018
@dnhatn
AwaitsFix IndexShardTests#testDocStats
139631c 
Relates #32449
dnhatn added a commit that referenced this issue Jul 28, 2018
@dnhatn
AwaitsFix IndexShardTests#testDocStats
1e7d0a8 
Relates #32449
@dnhatn
Copy link
Member Author

dnhatn commented Jul 28, 2018

I've muted this test.

dnhatn added a commit to dnhatn/elasticsearch that referenced this issue Jul 28, 2018
@dnhatn
TEST: testDocStats should always use forceMerge
99014e7 
Due to the recent change in LUCENE-8263, we need to adjust the deletion
ration to between 10% to 33% to preserve the current behavior of the
test. However, we may need another refinement if soft-deletes is enabled
as the actual deletes are different because of delete tombstones.

This commit prefers to always execute forceMerge instead of adjusting
the deletion ratio so that this test can focus on testing docStats.

Closes elastic#32449
@dnhatn dnhatn mentioned this issue Jul 28, 2018
Merged
dnhatn added a commit that referenced this issue Jul 28, 2018
@dnhatn
TEST: testDocStats should always use forceMerge (#32450)
5b1ad80 
Due to the recent change in LUCENE-8263, we need to adjust the deletion
ration to between 10% to 33% to preserve the current behavior of the
test. However, we may need another refinement if soft-deletes is enabled
as the actual deletes are different because of delete tombstones.

This commit prefers to always execute forceMerge instead of adjusting
the deletion ratio so that this test can focus on testing docStats.

Closes #32449
dnhatn added a commit that referenced this issue Jul 28, 2018
@dnhatn
TEST: testDocStats should always use forceMerge (#32450)
c6a092d 
Due to the recent change in LUCENE-8263, we need to adjust the deletion
ration to between 10% to 33% to preserve the current behavior of the
test. However, we may need another refinement if soft-deletes is enabled
as the actual deletes are different because of delete tombstones.

This commit prefers to always execute forceMerge instead of adjusting
the deletion ratio so that this test can focus on testing docStats.

Closes #32449
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
:Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@dnhatn @elasticmachine