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
Automated cherry pick of #97860: move all variables in sampleAndWaterMarkHistograms::innerSet #98432
Automated cherry pick of #97860: move all variables in sampleAndWaterMarkHistograms::innerSet #98432
Conversation
/kind bug |
/assign @MikeSpreitzer @deads2k @lavalamp |
/lgtm |
/priority important-soon |
/triage accepted |
/retest |
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.
/lgtm
@tkashem parent PR merged to master
18 days ago so I am guessing this needs CP on 1.20 also?
/hold
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: hasheddan, MikeSpreitzer, tkashem The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@hasheddan 1.20 cherry-pick has already merged - #97866 |
@tkashem okay 👍 Please do your best to group cherry picks together when they need to go to multiple branches. This means the fix was in v1.20.2, but not v1.19.7, which went out the same day. Although from the context, it looks like it may not have been realized at the time that this was applicable to 1.19. /hold cancel |
Cherry pick of #97860 on release-1.19.
#97860: move all variables in sampleAndWaterMarkHistograms::innerSet
Reason for cherry pick: Fixes #97685
Does this PR introduce a user-facing change?: