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

Backport #8701 #8705

Merged
merged 2 commits into from
Aug 15, 2017
Merged

Backport #8701 #8705

merged 2 commits into from
Aug 15, 2017

Conversation

jwilder
Copy link
Contributor

@jwilder jwilder commented Aug 15, 2017

Required for all non-trivial PRs
  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)

Backport #8701

If there were multiple shards, drop measurement could update the index
and remove the measurement before the other shards ran their deletes.
This causes the later shards to not see any series to delete.

The fix is to all deleteSeries to handle the index delete which already
accounts for removing the measurement when it is fully removed from the
index.
@jwilder jwilder added this to the 1.3.4 milestone Aug 15, 2017
@stuartcarnie stuartcarnie self-requested a review August 15, 2017 21:10
Copy link
Contributor

@stuartcarnie stuartcarnie left a comment

Choose a reason for hiding this comment

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

👍

@jwilder jwilder merged commit 14ab24f into 1.3 Aug 15, 2017
@jwilder jwilder deleted the jw-13-backports branch August 15, 2017 21:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants