Skip to content

Comments

Add more logging in vtgate buffering - ensureConsistentLocked (#771)#781

Closed
tanjinx wants to merge 3 commits intoslack-22.0from
more-vtgate-buffering-logging
Closed

Add more logging in vtgate buffering - ensureConsistentLocked (#771)#781
tanjinx wants to merge 3 commits intoslack-22.0from
more-vtgate-buffering-logging

Conversation

@tanjinx
Copy link

@tanjinx tanjinx commented Jan 27, 2026

Description

Related Issue(s)

Checklist

  • "Backport to:" labels have been added if this change should be back-ported to release branches
  • If this change is to be back-ported to previous releases, a justification is included in the PR description
  • Tests were added or are not required
  • Did the new or modified tests pass consistently locally and on CI?
  • Documentation was added or is not required

Deployment Notes

AI Disclosure

@tanjinx tanjinx requested a review from a team as a code owner January 27, 2026 06:52
@github-actions github-actions bot added this to the v22.0.2 milestone Jan 27, 2026
@codecov-commenter
Copy link

codecov-commenter commented Jan 27, 2026

Codecov Report

❌ Patch coverage is 54.54545% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.74%. Comparing base (b1ab347) to head (17c75c8).
⚠️ Report is 24 commits behind head on slack-22.0.

Files with missing lines Patch % Lines
go/vt/discovery/keyspace_events.go 54.54% 5 Missing ⚠️
Additional details and impacted files
@@              Coverage Diff               @@
##           slack-22.0     #781      +/-   ##
==============================================
+ Coverage       67.53%   69.74%   +2.21%     
==============================================
  Files            1600     1605       +5     
  Lines          261782   213571   -48211     
==============================================
- Hits           176786   148950   -27836     
+ Misses          84996    64621   -20375     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@tanjinx tanjinx closed this Feb 18, 2026
@tanjinx tanjinx deleted the more-vtgate-buffering-logging branch February 18, 2026 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants