-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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:fix retrieve raw data from bitmap array index #34848
fix:fix retrieve raw data from bitmap array index #34848
Conversation
Signed-off-by: luzhang <luzhang@zilliz.com>
rerun ut |
1 similar comment
rerun ut |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #34848 +/- ##
===========================================
- Coverage 84.44% 71.69% -12.76%
===========================================
Files 892 1170 +278
Lines 117033 142444 +25411
===========================================
+ Hits 98832 102122 +3290
- Misses 13832 35899 +22067
- Partials 4369 4423 +54
|
rerun ut |
@@ -44,6 +44,8 @@ const ( | |||
|
|||
// Sparse Index Param | |||
SparseDropRatioBuild = "drop_ratio_build" | |||
|
|||
MaxBitmapCardinalityLimit = 1000 |
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.
Should this be made configurable?
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: czs007, zhagnlu 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 |
rerun ut |
milvus-io#34795 Signed-off-by: luzhang <luzhang@zilliz.com> Co-authored-by: luzhang <luzhang@zilliz.com> Signed-off-by: Sumit Dubey <sumit.dubey2@ibm.com>
#34795