-
Notifications
You must be signed in to change notification settings - Fork 2.7k
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 error ignore in compactor #34169
Conversation
Signed-off-by: wayblink <anyang.wang@zilliz.com>
@wayblink Please associate the related issue to the body of your Pull Request. (eg. “issue: #”) |
/lgtm |
@wayblink E2e jenkins job failed, comment |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #34169 +/- ##
==========================================
- Coverage 84.65% 81.00% -3.66%
==========================================
Files 809 1077 +268
Lines 113143 137177 +24034
==========================================
+ Hits 95779 111117 +15338
- Misses 13147 21845 +8698
+ Partials 4217 4215 -2
|
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: czs007, wayblink 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 |
milvus-io#34170 Signed-off-by: wayblink <anyang.wang@zilliz.com>
milvus-io#34170 Signed-off-by: wayblink <anyang.wang@zilliz.com>
milvus-io#34170 Signed-off-by: wayblink <anyang.wang@zilliz.com>
milvus-io#34170 Signed-off-by: wayblink <anyang.wang@zilliz.com>
milvus-io#34170 Signed-off-by: wayblink <anyang.wang@zilliz.com>
This PR cherry-picks the following commits: - fix: sync partitiion stats blocking balance task #33742 - fix: Fix meta prefix overlap bug #33830 - fix: Small fixs of major compaction #33929 - fix: Fix memory buffer error & some renaming #33850 - fix: sync part stats task cannot be finished #34027 - Add an option to enable/disable vector field clustering key #34097 - fix: fix error ignore in compactor #34169 - fix:load major compaction partial result #34052 - Use new stream segment reader in clustering compaction #34232 issue: #30633 pr: #33742 #33830 #33929 #33850 #34027 #34097 #34169 #34052 #34232 --------- Signed-off-by: MrPresent-Han <chun.han@zilliz.com> Signed-off-by: wayblink <anyang.wang@zilliz.com> Signed-off-by: MrPresent-Han <chun.han@gmail.com> Co-authored-by: Chun Han <116052805+MrPresent-Han@users.noreply.github.com> Co-authored-by: MrPresent-Han <chun.han@gmail.com>
#34170