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

streamingccl: make MVCC bulk operations well-tested #83810

Closed
shermanCRL opened this issue Jul 5, 2022 · 3 comments · Fixed by #84553
Closed

streamingccl: make MVCC bulk operations well-tested #83810

shermanCRL opened this issue Jul 5, 2022 · 3 comments · Fixed by #84553
Assignees
Labels
A-tenant-streaming Including cluster streaming
Milestone

Comments

@shermanCRL
Copy link
Contributor

shermanCRL commented Jul 5, 2022

Jira issue: CRDB-17277

@blathers-crl
Copy link

blathers-crl bot commented Jul 5, 2022

Hi @shermanCRL, please add a C-ategory label to your issue. Check out the label system docs.

While you're here, please consider adding an A- label to help keep our repository tidy.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is otan.

@shermanCRL shermanCRL added the A-tenant-streaming Including cluster streaming label Jul 5, 2022
@blathers-crl
Copy link

blathers-crl bot commented Jul 5, 2022

cc @cockroachdb/tenant-streaming

@shermanCRL shermanCRL added this to the 22.2 milestone Jul 5, 2022
@gh-casper gh-casper self-assigned this Aug 1, 2022
@gh-casper gh-casper changed the title streamingccl: adopt MVCC bulk operations streamingccl: make MVCC bulk operations well-tested Aug 4, 2022
@gh-casper
Copy link
Contributor

Even though we now have test coverage in some unit tests and random stream client that generates SSTs and DelRanges, we still need to test bulk ops in roachtests and long-lived manual testing.
#76947
#83315

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-tenant-streaming Including cluster streaming
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants