Closed congqixia closed 13 hours ago
Attention: Patch coverage is 0%
with 1 line
in your changes missing coverage. Please review.
Project coverage is 81.02%. Comparing base (
c07f056
) to head (e52eac5
). Report is 16 commits behind head on master.
Files with missing lines | Patch % | Lines |
---|---|---|
internal/querynodev2/segments/segment.go | 0.00% | 1 Missing :warning: |
🚨 Try these New Features:
@congqixia E2e jenkins job failed, comment /run-cpu-e2e
can trigger the job again.
/run-cpu-e2e
Why to use a load pool for index building? That doesn't seems to be a good fix. /hold
@xiaofan-luan it was building temporary text index in load process. At least it shall not use dynamic pool. Don't think it's a good idea to open other pool in querynode.
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: congqixia, xiaofan-luan
The full list of commands accepted by this bot can be found here.
The pull request process is described here
/unhold
Related to #37895
Only resolves the starving issue which caused goroutine leakage