Athul0491 commented on issue #22419:
URL: https://github.com/apache/datafusion/issues/22419#issuecomment-4911655183
Hi. I’d like to take a small, scoped part of this issue.
My plan is as follows:
- add a feature-gated try_to_proto hook on ExecutionPlan, defaulting to
Ok(None) so existing behavior stays unchanged
- add the matching decode-side plumbing so migrated plan types can define
try_from_proto(...)
- migrate one built-in plan as the first example, likely FilterExec
- keep the current central downcast/match path in place for everything
else for now
That should establish the pattern without mixing in the broader migration
and cleanup. If that sounds right, I can
open a PR for the hook plumbing plus the first migrated plan.
--
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]
