fix: use seperate warmup pool and disable warmup by default #33348
Conversation
Signed-off-by: bigsheeper <[email protected]> Co-authored-by: xiaofanluan <[email protected]>
mergify
bot
added
dco-passed
|
@bigsheeper Please associate the related issue to the body of your Pull Request. (eg. “issue: #”) |
Signed-off-by: bigsheeper <[email protected]>
Signed-off-by: bigsheeper <[email protected]>
Signed-off-by: bigsheeper <[email protected]>
Signed-off-by: bigsheeper <[email protected]>
|
@bigsheeper E2e jenkins job failed, comment |
|
/run-cpu-e2e |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #33348 +/- ##
=======================================
Coverage 82.12% 82.13%
=======================================
Files 1012 1012
Lines 128851 128872 +21
=======================================
+ Hits 105819 105847 +28
+ Misses 19054 19036 -18
- Partials 3978 3989 +11
|
|
/approve |
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bigsheeper, czs007 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 |
…33349) 1. use a small warmup pool to reduce the impact of warmup 2. change the warmup pool to nonblocking mode 3. disable warmup by default 4. remove the maximum size limit of 16 for the load pool issue: #32772 pr: #33348 --------- Signed-off-by: bigsheeper <[email protected]> Co-authored-by: xiaofanluan <[email protected]>
issue: #32772