-
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
clusterversion: remove TODOPreV22_1 #96701
Conversation
f78ceef
to
ffcfff2
Compare
ffcfff2
to
33260f4
Compare
This change removes this constant and the obsolete code that uses it. Release note: None Epic: none
33260f4
to
16878ed
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reviewed 2 of 10 files at r1.
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @celiala and @jayshrivastava)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TFTR! I'll merge since the only non-storage changes are removal of two tiny dead code sections.
bors r+
Reviewable status: complete! 1 of 0 LGTMs obtained (waiting on @celiala and @jayshrivastava)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!! LGTM from the release eng side
CDC side LGTM! |
Build failed (retrying...): |
Build succeeded: |
This change removes this constant and the obsolete code that uses it.
Release note: None
Epic: None