-
Notifications
You must be signed in to change notification settings - Fork 3k
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
fix: [2.4] Fix timeout when listing meta #38216
base: 2.4
Are you sure you want to change the base?
fix: [2.4] Fix timeout when listing meta #38216
Conversation
Signed-off-by: bigsheeper <[email protected]>
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: bigsheeper The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@bigsheeper E2e jenkins job failed, comment |
/run-cpu-e2e |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## 2.4 #38216 +/- ##
===========================================
+ Coverage 70.40% 80.58% +10.17%
===========================================
Files 1074 1074
Lines 168099 168110 +11
===========================================
+ Hits 118353 135471 +17118
+ Misses 45291 28205 -17086
+ Partials 4455 4434 -21
|
rerun ut |
When there are too many key-value pairs, the etcd list operation may times out. This PR replaces LoadWithPrefix in list operations, which could involve many keys, with WalkWithPrefix.
issue: #37917
pr: #38151