adriangb commented on issue #23494: URL: https://github.com/apache/datafusion/issues/23494#issuecomment-5028370917
Guidance refinement: the deprecated `PhysicalPlanNodeExt` shims should now **delegate** to the plan's new `try_to_proto` / `try_from_proto` hook rather than keep their original duplicated bodies (this supersedes the "keep them with their original bodies" note in my 2026-07-17 comment above). `FilterExec` (#23708) already follows this, and #23731 updates the `ProjectionExec` reference to match. Deleting the old central encode `downcast_ref` arm + repointing the decode arm in the same PR is still required. -- 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]
