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

FlushIT.testFlushOnInactive failures #52251

Closed
jkakavas opened this issue Feb 12, 2020 · 2 comments · Fixed by #52275
Closed

FlushIT.testFlushOnInactive failures #52251

jkakavas opened this issue Feb 12, 2020 · 2 comments · Fixed by #52275
Assignees
@dnhatn
Labels
:Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >test-failure Triaged test failures from CI v8.0.0-alpha1

Comments

@jkakavas
Copy link
Member

This failed 4 times in the last four days in windows and debian in master. There were no recent changes in that class, maybe @dnhatn has an idea what is the issue here. I'll mute this as a precaution. In all cases the number of uncommitted translog entries were not 0 as expected.

Example failures:
https://gradle-enterprise.elastic.co/s/f52glzd6m2jtk
https://gradle-enterprise.elastic.co/s/7brl5xuyklpia
@jkakavas jkakavas added :Core/Infra/Core Core issues without another label >test-failure Triaged test failures from CI v8.0.0 labels Feb 12, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-core-infra (:Core/Infra/Core)

@jkakavas jkakavas changed the title FlusIT.testFlushOnInactive failures FlushIT.testFlushOnInactive failures Feb 12, 2020
@original-brownbear original-brownbear added :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. and removed :Core/Infra/Core Core issues without another label labels Feb 12, 2020
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-distributed (:Distributed/Engine)

@jkakavas jkakavas mentioned this issue Feb 12, 2020
Merged
jkakavas added a commit that referenced this issue Feb 12, 2020
@jkakavas
Mute FlushIT.testFlushOnInactive (#52252)
aa0e882 
See #52251
@dnhatn dnhatn self-assigned this Feb 12, 2020
@dnhatn dnhatn mentioned this issue Feb 12, 2020
Merged
dnhatn added a commit that referenced this issue Feb 12, 2020
@dnhatn
Fix testFlushOnInactive (#52275)
be4562b 
We need to reduce the translog sync interval for indices with translog 
async setting so that we can have the safe commit in the assertBusy
interval. This is needed since #51905, where we use the local checkpoint
of the safe commit to calculate the number of uncommitted operations of
a translog stats.

Closes #52251
Relates #51905
dnhatn added a commit that referenced this issue Feb 12, 2020
@dnhatn
Fix testFlushOnInactive (#52275)
12cb6dc 
We need to reduce the translog sync interval for indices with translog
async setting so that we can have the safe commit in the assertBusy
interval. This is needed since #51905, where we use the local checkpoint
of the safe commit to calculate the number of uncommitted operations of
a translog stats.

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

@dnhatn dnhatn

Labels
:Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. >test-failure Triaged test failures from CI v8.0.0-alpha1
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fix testFlushOnInactive dnhatn/elasticsearch
5 participants
@jakelandis @original-brownbear @jkakavas @dnhatn @elasticmachine