-
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
fix: Skip load extra indexes for sorted segment pk field #39389
base: master
Are you sure you want to change the base?
fix: Skip load extra indexes for sorted segment pk field #39389
Conversation
Related to milvus-io#39339 Extra indexes can be ignored for most cases since sorted pk column already provided indexing features Signed-off-by: Congqi Xia <[email protected]>
[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
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
@congqixia go-sdk check failed, comment |
rerun go-sdk |
Signed-off-by: Congqi Xia <[email protected]>
@congqixia go-sdk check failed, comment |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #39389 +/- ##
==========================================
+ Coverage 80.99% 81.03% +0.04%
==========================================
Files 1407 1407
Lines 198528 198532 +4
==========================================
+ Hits 160794 160886 +92
+ Misses 32063 31998 -65
+ Partials 5671 5648 -23
|
rerun go-sdk |
Cherry-pick from master pr: #39389 Related to #39339 Extra indexes can be ignored for most cases since sorted pk column already provided indexing features --------- Signed-off-by: Congqi Xia <[email protected]>
Related to #39339
Extra indexes can be ignored for most cases since sorted pk column already provided indexing features