milvus
milvus copied to clipboard
fix: Add Wrapper and Keepalive for CTraceContext ids
See also #32742
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: congqixia
The full list of commands accepted by this bot can be found here.
The pull request process is described here
- ~~internal/querynodev2/segments/OWNERS~~ [congqixia]
Approvers can indicate their approval by writing /approve
in a comment
Approvers can cancel approval by writing /approve cancel
in a comment
@congqixia E2e jenkins job failed, comment /run-cpu-e2e
can trigger the job again.
/run-cpu-e2e
@congqixia E2e jenkins job failed, comment /run-cpu-e2e
can trigger the job again.
@congqixia ut workflow job failed, comment rerun ut
can trigger the job again.
@congqixia E2e jenkins job failed, comment /run-cpu-e2e
can trigger the job again.
Codecov Report
Attention: Patch coverage is 87.50000%
with 2 lines
in your changes are missing coverage. Please review.
Project coverage is 81.72%. Comparing base (
600db9d
) to head (ea62399
). Report is 2 commits behind head on master.
Additional details and impacted files
@@ Coverage Diff @@
## master #32746 +/- ##
==========================================
- Coverage 81.77% 81.72% -0.05%
==========================================
Files 1001 1001
Lines 125359 125365 +6
==========================================
- Hits 102508 102454 -54
- Misses 18966 19021 +55
- Partials 3885 3890 +5
Files | Coverage Δ | |
---|---|---|
internal/querynodev2/segments/load_index_info.go | 73.33% <100.00%> (+0.19%) |
:arrow_up: |
internal/querynodev2/segments/trace.go | 100.00% <100.00%> (ø) |
|
internal/querynodev2/segments/segment.go | 67.08% <66.66%> (-0.01%) |
:arrow_down: |
/run-cpu-e2e
/lgtm
@congqixia E2e jenkins job failed, comment /run-cpu-e2e
can trigger the job again.
/run-cpu-e2e
@congqixia E2e jenkins job failed, comment /run-cpu-e2e
can trigger the job again.
/lgtm