Skip to content

[QueryThrottler] Implement Query Throttler#18449

Merged
shlomi-noach merged 34 commits into
vitessio:mainfrom
siddharth16396:throttler-first
Sep 15, 2025
Merged

[QueryThrottler] Implement Query Throttler#18449
shlomi-noach merged 34 commits into
vitessio:mainfrom
siddharth16396:throttler-first

remove duplicate comment

13cb472
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project succeeded Sep 14, 2025 in 0s

67.54% (+0.03%) compared to 770ae74

View this Pull Request on Codecov

67.54% (+0.03%) compared to 770ae74

Details

Codecov Report

❌ Patch coverage is 90.05525% with 18 lines in your changes missing coverage. Please review.
✅ Project coverage is 67.54%. Comparing base (770ae74) to head (13cb472).
⚠️ Report is 10 commits behind head on main.

Files with missing lines Patch % Lines
...let/tabletserver/querythrottler/query_throttler.go 88.31% 9 Missing ⚠️
go/vt/vttablet/tabletserver/query_executor.go 33.33% 4 Missing ⚠️
...t/tabletserver/querythrottler/registry/registry.go 92.50% 3 Missing ⚠️
go/vt/vttablet/tabletserver/tabletserver.go 80.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main   #18449      +/-   ##
==========================================
+ Coverage   67.51%   67.54%   +0.03%     
==========================================
  Files        1607     1613       +6     
  Lines      263357   263742     +385     
==========================================
+ Hits       177815   178155     +340     
- Misses      85542    85587      +45     

☔ 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.