-
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
enhance: use configed max topk for iterator when input topk exceeds(#34292) #34293
enhance: use configed max topk for iterator when input topk exceeds(#34292) #34293
Conversation
f7df327
to
8a7bf17
Compare
8a7bf17
to
be2af71
Compare
@MrPresent-Han E2e jenkins job failed, comment |
/run-cpu-e2e |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## 2.4 #34293 +/- ##
==========================================
- Coverage 84.47% 81.02% -3.45%
==========================================
Files 776 1034 +258
Lines 111721 134836 +23115
==========================================
+ Hits 94379 109255 +14876
- Misses 13143 21380 +8237
- Partials 4199 4201 +2
|
/run-cpu-e2e |
…ilvus-io#34292) Signed-off-by: MrPresent-Han <chun.han@gmail.com>
be2af71
to
047073f
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: czs007, MrPresent-Han 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 |
related: #34292
pr: #34290