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

Update translog policy before the next safe commit #54839

Merged
merged 3 commits into from
Apr 7, 2020

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Apr 6, 2020

IndexShardIT#testMaybeFlush relies on the assumption that once IndexShard#sync returns, we have advanced the safe commit and translog deletion policy. This assumption does not hold if there's a race with the global checkpoint sync.

Closes #52223

@dnhatn dnhatn added >bug :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. v8.0.0 v7.8.0 v7.7.1 labels Apr 6, 2020
@dnhatn dnhatn requested a review from ywelsch April 6, 2020 19:14
@elasticmachine
Copy link
Collaborator

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

@dnhatn
Copy link
Member Author

dnhatn commented Apr 6, 2020

run elasticsearch-ci/packaging-sample-unix-docker

Copy link
Contributor

@ywelsch ywelsch left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@dnhatn
Copy link
Member Author

dnhatn commented Apr 7, 2020

Thanks Yannick.

@dnhatn dnhatn merged commit 3bfcc60 into elastic:master Apr 7, 2020
@dnhatn dnhatn deleted the fix-maybe-flush branch April 7, 2020 15:14
dnhatn added a commit that referenced this pull request Apr 8, 2020
IndexShardIT#testMaybeFlush relies on the assumption that the safe commit
and translog deletion policy have advanced after IndexShard#sync returns .
This assumption does not hold if there's a race with the global checkpoint sync.

Closes #52223
dnhatn added a commit that referenced this pull request Apr 8, 2020
IndexShardIT#testMaybeFlush relies on the assumption that the safe commit
and translog deletion policy have advanced after IndexShard#sync returns .
This assumption does not hold if there's a race with the global checkpoint sync.

Closes #52223
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>bug :Distributed Indexing/Engine Anything around managing Lucene and the Translog in an open shard. v7.7.1 v7.8.0 v8.0.0-alpha1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] IndexShardIT » testMaybeFlush
4 participants