-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
release-23.1: kvcoord: Use locked map in mux rangefeed #99469
Conversation
Use locked map instead of IntMap to ensure correct synchronization between stream start up and termination. Fixes #99096 Release note: None Release justficiation: bug fix
1feae80
to
374b4a3
Compare
Thanks for opening a backport. Please check the backport criteria before merging:
If some of the basic criteria cannot be satisfied, ensure that the exceptional criteria are satisfied within.
Add a brief release justification to the body of your PR to justify this backport. Some other things to consider:
|
It looks like your PR touches production code but doesn't add or edit any test code. Did you consider adding tests to your PR? 🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf. |
bors r+ |
This PR was included in a batch that successfully built, but then failed to merge into release-23.1 (it was a non-fast-forward update). It will be automatically retried. |
Build succeeded: |
Backport 1/1 commits from #99289 on behalf of @miretskiy.
/cc @cockroachdb/release
Use locked map instead of IntMap to ensure correct
synchronization between stream start up and termination.
Fixes #99096
Release note: None
Release justification: bug fix
Release justification: