-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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: kvserver: fix and re-enable slow lease application logging #105532
Conversation
Epic: none Release note: None
bcbc49f
to
4d38240
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:
|
e640081
to
0de16c1
Compare
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. |
Will let this sit for a bit, to make sure it doesn't result in a bunch of log spam. |
Reminder: it has been 3 weeks please merge or close your backport! |
Going to poke at some roachtests this week, will have a look at this. |
Reminder: it has been 3 weeks please merge or close your backport! |
@erikgrinaker do we want to merge this? |
I think it's fine to leave this for 23.2. |
Backport 1/1 commits from #105519 on behalf of @erikgrinaker.
/cc @cockroachdb/release
Resolves #97209.
Epic: none
Release note: None
Release justification: