-
Notifications
You must be signed in to change notification settings - Fork 322
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
feat: upgrade third-party to 0.6.1 #3736
Conversation
u need upgrade in third-party/ first: OpenMLDB/third-party/CMakeLists.txt Lines 70 to 71 in b2f85fb
|
SDK Test Report101 files ±0 101 suites ±0 2m 4s ⏱️ -2s Results for commit c7bac8f. ± Comparison against base commit 7d755ff. This pull request removes 48 and adds 27 tests. Note that renamed tests count towards both.
This pull request skips 1 test.
♻️ This comment has been updated with latest results. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3736 +/- ##
=============================================
- Coverage 74.91% 40.09% -34.83%
- Complexity 655 657 +2
=============================================
Files 732 193 -539
Lines 131613 11414 -120199
Branches 1366 1374 +8
=============================================
- Hits 98595 4576 -94019
+ Misses 32719 6535 -26184
- Partials 299 303 +4 ☔ View full report in Codecov by Sentry. |
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
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
What is the new behavior (if this is a feature change)?