Skip to content

Account soft deletes for committed segments#43126

Merged
dnhatn merged 1 commit intoelastic:masterfrom
dnhatn:soft-del-segment
Jun 14, 2019
Merged

Account soft deletes for committed segments#43126
dnhatn merged 1 commit intoelastic:masterfrom
dnhatn:soft-del-segment

Conversation

@dnhatn
Copy link
Copy Markdown
Member

@dnhatn dnhatn commented Jun 11, 2019

This change fixes the delete count issue in segment stats where we don't account soft-deleted documents from committed segments.

Relates #43103

@dnhatn dnhatn added >bug :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v8.0.0 v7.2.0 v7.3.0 v6.8.1 v7.1.2 labels Jun 11, 2019
@dnhatn dnhatn requested review from jpountz and ywelsch June 11, 2019 21:51
@elasticmachine
Copy link
Copy Markdown
Collaborator

Pinging @elastic/es-distributed

Copy link
Copy Markdown
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. Can you follow-up with a PR that exposes stats about soft-deleted docs?

@dnhatn
Copy link
Copy Markdown
Member Author

dnhatn commented Jun 13, 2019

@ywelsch Thanks for reviewing. I've responded on the issue.

@dnhatn dnhatn merged commit fdacbaf into elastic:master Jun 14, 2019
@dnhatn dnhatn deleted the soft-del-segment branch June 14, 2019 16:43
dnhatn added a commit that referenced this pull request Jun 15, 2019
This change fixes the delete count issue in segment stats where we don't
account soft-deleted documents from committed segments.

Relates #43103
dnhatn added a commit that referenced this pull request Jun 15, 2019
This change fixes the delete count issue in segment stats where we don't
account soft-deleted documents from committed segments.

Relates #43103
dnhatn added a commit that referenced this pull request Jun 15, 2019
This change fixes the delete count issue in segment stats where we don't
account soft-deleted documents from committed segments.

Relates #43103
dnhatn added a commit that referenced this pull request Jun 17, 2019
This change fixes the delete count issue in segment stats where we don't
account soft-deleted documents from committed segments.

Relates #43103
kovrus added a commit to crate/crate that referenced this pull request Sep 3, 2019
Backport of elastic/elasticsearch#43126

This change fixes the delete count issue in segment stats where
soft-deleted documents from committed segments are not accounted.
@mfussenegger mfussenegger mentioned this pull request Mar 26, 2020
37 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

>bug :Distributed/Engine Anything around managing Lucene and the Translog in an open shard. v6.8.1 v7.1.2 v7.2.0 v7.3.0 v8.0.0-alpha1

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants