buraksenn opened a new pull request, #24189: URL: https://github.com/apache/datafusion/pull/24189
## Which issue does this PR close? - Part of #23516. ## Rationale for this change Part of epic #23494. Moves `ArrowSource` protobuf serialization from the central dispatch into the source implementation. ## What changes are included in this PR? Add protobuf serialization and deserialization to `ArrowSource` and add the required `proto` feature wiring to `datafusion-datasource-arrow`. Repoint the live decode arm to `ArrowSource::try_from_proto` and remove the old central encode arm. Keep `try_into_arrow_scan_physical_plan` as a deprecated compatibility wrapper that delegates to the new implementation. The protobuf wire format remains unchanged. ## Are these changes tested? Yes. The existing `roundtrip_arrow_scan` coverage passes through the new hooks. ## Are there any user-facing changes? The existing `PhysicalPlanNodeExt` method remains available as a deprecated compatibility wrapper. There is no immediate API removal or wire-format change. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
