-
Notifications
You must be signed in to change notification settings - Fork 3.3k
fix: use tantivy version to make json index compatible with milvus 2.5 #43563
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
Conversation
Signed-off-by: sunby <[email protected]>
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## master #43563 +/- ##
=======================================
Coverage 78.96% 78.96%
=======================================
Files 1567 1567
Lines 224630 224644 +14
=======================================
+ Hits 177374 177399 +25
+ Misses 40831 40821 -10
+ Partials 6425 6424 -1
🚀 New features to boost your workflow:
|
field_name.c_str(), | ||
this->d_type_, | ||
this->path_.c_str(), | ||
TANTIVY_INDEX_LATEST_VERSION /* json index is not supported in old version */); |
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.
Is json index supported in 2.5?
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.
Is json index supported in 2.5?
actually yes
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
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: SpadeA-Tang, sunby, zhengbuqian 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 |
issue: #43562