-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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 inconsistency in DataFrameColumns Clone API implementation #7100
Fix inconsistency in DataFrameColumns Clone API implementation #7100
Conversation
… make base Clone method not virtual and CloneImplementation abstract
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #7100 +/- ##
==========================================
- Coverage 68.46% 68.46% -0.01%
==========================================
Files 1263 1263
Lines 254956 254945 -11
Branches 26352 26353 +1
==========================================
- Hits 174552 174542 -10
+ Misses 73698 73697 -1
Partials 6706 6706
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.
Looks good to me. Can you resolve the conflicts @asmirnov82? They might have come from me just merging in your other PR.
# Conflicts: # src/Microsoft.Data.Analysis/DataFrameColumns/ArrowStringDataFrameColumn.cs # src/Microsoft.Data.Analysis/PrimitiveDataFrameColumn.cs
Thanks, Michael. Conflicts are resolved now |
There are no breaking changes on Public API, mosty only internal implementation is changed
Fixes #7092