feat: publish index artifacts through transactions - #620
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: yuruiz The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @yuruiz! It looks like this is your first PR to milvus-io/milvus-storage 🎉 |
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #620 +/- ##
==========================================
+ Coverage 76.19% 76.42% +0.23%
==========================================
Files 173 173
Lines 17388 17561 +173
Branches 2618 2655 +37
==========================================
+ Hits 13248 13421 +173
Misses 4140 4140
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
Addressed the latest review comments in commit
Validation: legacy v2/v3 tests passed and |
|
/lgtm |
Summary
Transaction::AddIndexInfoIndexLoonIndexInfoand C FFIManifestMinorVersion; index presence is represented by manifestindexesTesting
milvus_testandTest_FFITest_FFI(78 tests, 0 failed)