-
Notifications
You must be signed in to change notification settings - Fork 141
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 core refactor: StreamIO from common to core.common (#296) #1869
Fix core refactor: StreamIO from common to core.common (#296) #1869
Conversation
* Fix core refactor: StreamIO from common to core.common Signed-off-by: acarbonetto <[email protected]> * Fix core refactor: StreamIO from common to core.common Signed-off-by: acarbonetto <[email protected]> --------- Signed-off-by: acarbonetto <[email protected]>
372ef91
to
0a2f6d7
Compare
Codecov Report
@@ Coverage Diff @@
## main #1869 +/- ##
=========================================
Coverage 97.39% 97.39%
Complexity 4602 4602
=========================================
Files 401 401
Lines 11386 11386
Branches 843 843
=========================================
Hits 11089 11089
Misses 290 290
Partials 7 7
Flags with carried forward coverage won't be shown. Click here to find out more.
|
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.
Ignore BWC failures
…arch-project#1869) * Fix core refactor: StreamIO from common to core.common * Fix core refactor: StreamIO from common to core.common --------- Signed-off-by: acarbonetto <[email protected]>
…arch-project#1869) * Fix core refactor: StreamIO from common to core.common * Fix core refactor: StreamIO from common to core.common --------- Signed-off-by: acarbonetto <[email protected]> Signed-off-by: Mitchell Gale <[email protected]>
…arch-project#1869) * Fix core refactor: StreamIO from common to core.common * Fix core refactor: StreamIO from common to core.common --------- Signed-off-by: acarbonetto <[email protected]> Signed-off-by: Mitchell Gale <[email protected]>
* Fix core refactor: StreamIO from common to core.common * Fix core refactor: StreamIO from common to core.common --------- Signed-off-by: acarbonetto <[email protected]> (cherry picked from commit dae5a85)
…#296) (#1891) * Fix core refactor: StreamIO from common to core.common (#296) (#1869) * Fix core refactor: StreamIO from common to core.common * Fix core refactor: StreamIO from common to core.common --------- Signed-off-by: acarbonetto <[email protected]> (cherry picked from commit dae5a85) * Bump Version and Fix import error Signed-off-by: Vamsi Manohar <[email protected]> --------- Signed-off-by: Vamsi Manohar <[email protected]> Co-authored-by: Andrew Carbonetto <[email protected]> Co-authored-by: Vamsi Manohar <[email protected]>
…arch-project#1869) * Fix core refactor: StreamIO from common to core.common * Fix core refactor: StreamIO from common to core.common --------- Signed-off-by: acarbonetto <[email protected]> Signed-off-by: Mitchell Gale <[email protected]>
Description
Fix compilation errors after opensearch-project/OpenSearch#8157.
Note: May need backport 2.x label.
In this PR it's only changing package name of core classes in import section
Issues Resolved
Fixes changes from opensearch-project/ml-commons#1122
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.