-
Notifications
You must be signed in to change notification settings - Fork 44.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 MilvusMemory connecting to Zilliz #3278
Fix MilvusMemory connecting to Zilliz #3278
Conversation
Codecov ReportPatch coverage has no change and project coverage change:
Additional details and impacted files@@ Coverage Diff @@
## master #3278 +/- ##
==========================================
- Coverage 51.22% 43.25% -7.98%
==========================================
Files 65 65
Lines 2975 2973 -2
Branches 502 502
==========================================
- Hits 1524 1286 -238
- Misses 1331 1612 +281
+ Partials 120 75 -45
☔ View full report in Codecov by Sentry. |
@Pwuts Hi, It's the bugfix commit for the fatal bug introduced by #2127. |
Hi, @Pwuts, Could you please review and fix this fatal error? :) |
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Background
A new bug was introduced by #2127
Changes
fix new bug in MilvusMemory construction.
Documentation
Test Plan
PR Quality Checklist