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

storage: revert experimental MVCC range tombstones #76921

Merged
merged 3 commits into from
Feb 23, 2022

Conversation

erikgrinaker
Copy link
Contributor

This reverts most of #76131, #76203, and #76478 -- except minor changes that were unrelated to the range tombstones themselves.

This leaves a gap for cluster version Internal:78 -- I think that's probably fine, but I've left a comment.

@erikgrinaker erikgrinaker requested review from a team as code owners February 23, 2022 09:28
@erikgrinaker erikgrinaker self-assigned this Feb 23, 2022
@cockroach-teamcity
Copy link
Member

This change is Reviewable

Copy link
Contributor

@aliher1911 aliher1911 left a comment

Choose a reason for hiding this comment

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

Nice git stash we got.

@erikgrinaker
Copy link
Contributor Author

Nice git stash we got.

There's plenty more where that came from.

@erikgrinaker
Copy link
Contributor Author

CI failure is unrelated flake.

bors r=aliher1911

@craig
Copy link
Contributor

craig bot commented Feb 23, 2022

Build succeeded:

@craig craig bot merged commit d35ac3a into cockroachdb:master Feb 23, 2022
@erikgrinaker erikgrinaker deleted the revert-mvcc-range-tombstones branch March 26, 2022 18:29
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.

3 participants