Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

enhance: decrease CPU overhead when calculating index file size #36579

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

jaime0815
Copy link
Contributor

issue: #36578

Signed-off-by: jaime <yun.zhang@zilliz.com>
@sre-ci-robot sre-ci-robot added the size/M Denotes a PR that changes 30-99 lines. label Sep 27, 2024
@sre-ci-robot
Copy link
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: jaime0815
To complete the pull request process, please assign xiaofan-luan after the PR has been reviewed.
You can assign the PR to them by writing /assign @xiaofan-luan in a comment when ready.

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 /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@mergify mergify bot added dco-passed DCO check passed. kind/enhancement Issues or changes related to enhancement labels Sep 27, 2024
Signed-off-by: jaime <yun.zhang@zilliz.com>
@mergify mergify bot added the ci-passed label Sep 27, 2024
Copy link

codecov bot commented Sep 27, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 73.34%. Comparing base (c056620) to head (e75997c).
Report is 10 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master   #36579      +/-   ##
==========================================
- Coverage   82.32%   73.34%   -8.98%     
==========================================
  Files        1279     1281       +2     
  Lines      151260   151436     +176     
==========================================
- Hits       124519   111077   -13442     
- Misses      21737    35333   +13596     
- Partials     5004     5026      +22     
Files with missing lines Coverage Δ
internal/datacoord/index_meta.go 95.83% <100.00%> (+0.08%) ⬆️
internal/datacoord/meta.go 89.74% <100.00%> (-0.07%) ⬇️

... and 266 files with indirect coverage changes

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
ci-passed dco-passed DCO check passed. kind/enhancement Issues or changes related to enhancement size/M Denotes a PR that changes 30-99 lines.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants