feat(python): support merge_columns via Ray integration
Expose merge_columns via Ray integration, and add one step to Commit the merge OP.
V2 does not support merge_column yet
https://github.com/lancedb/lance/issues/1929
I will resume this work once v2 merge_column support is done.
@eddyxu IMO I think we should use the same API as Dataset.add_columns() instead of merge_columns. It's similar but has a few more capabilities (like a SQL option). Also the merge_columns name is confusing because there is that older merge API that literally performs a join with pre-computed data.
Codecov Report
Attention: Patch coverage is 78.12041% with 149 lines in your changes are missing coverage. Please review.
Project coverage is 79.83%. Comparing base (
84d5d21) to head (be03a92).
:exclamation: Current head be03a92 differs from pull request most recent head a6a4837
Please upload reports for the commit a6a4837 to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## main #2278 +/- ##
=======================================
Coverage 79.82% 79.83%
=======================================
Files 202 202
Lines 55462 55462
Branches 55462 55462
=======================================
+ Hits 44274 44276 +2
Misses 8621 8621
+ Partials 2567 2565 -2
| Flag | Coverage Δ | |
|---|---|---|
| unittests | 79.83% <78.12%> (+<0.01%) |
:arrow_up: |
Flags with carried forward coverage won't be shown. Click here to find out more.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Any updates on this?