Add support for `CopyTo::partition_by` field in datafusion_proto
After https://github.com/apache/arrow-datafusion/pull/9240 datafusion protobuf will will silently ignore partition_by options;
The task is
-
add code to serialize / deserialize the
CopyTo::partition_byfield, added in https://github.com/apache/arrow-datafusion/pull/9240 to the existing serialization code (similarly to how copy_to is done) https://github.com/apache/arrow-datafusion/blob/f1dcbf978210be6aa69b07892cf47321905aeece/datafusion/proto/src/logical_plan/mod.rs#L850 -
Update the test in
datafusion/proto/tests/cases/roundtrip_logical_plan.rsto send a non empty partition column list
Originally posted by @devinjdangelo in https://github.com/apache/arrow-datafusion/pull/9240#discussion_r1491159044
Once https://github.com/apache/arrow-datafusion/pull/9240 is merged, I think this would be a good first issue as it will be small and self contained
I can work on this one.
I can work on this one.
Thanks @PhVHoang 🙏
https://github.com/apache/arrow-datafusion/pull/9240 was just merged so I think this one is now actionable
take