-
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
enhance: Remove l0 delete cache #32990
enhance: Remove l0 delete cache #32990
Conversation
ec7d956
to
acb5c5e
Compare
@xiaofan-luan ut workflow job failed, comment |
@xiaofan-luan E2e jenkins job failed, comment |
acb5c5e
to
3eb08ae
Compare
3eb08ae
to
8294d23
Compare
@xiaofan-luan ut workflow job failed, comment |
Signed-off-by: xiaofanluan <xiaofan.luan@zilliz.com>
8294d23
to
138a410
Compare
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #32990 +/- ##
==========================================
- Coverage 82.19% 82.15% -0.05%
==========================================
Files 1009 1009
Lines 128922 128797 -125
==========================================
- Hits 105970 105815 -155
- Misses 18963 18975 +12
- Partials 3989 4007 +18
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
[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
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
fix #32989
remove l0 cache and build delete pk and ts everytime. this reduce the memory and also increase the code readability