adriangb commented on issue #23494: URL: https://github.com/apache/datafusion/issues/23494#issuecomment-5005240566
The base PR (#23495) has merged, so the individual sub-tasks are now unblocked — anyone interested can pick one up! 🚀 Notes for contributors: - Follow the **Pattern (per plan)** section in the issue description above. `ProjectionExec` on main is the merged reference implementation; the freshly rebased end-state draft #23496 shows what every migration looks like against current main. - Per the recent review discussion on #23495, do **not** delete your plan's old `PhysicalPlanNodeExt` methods — keep them with their original bodies under `#[deprecated(since = "55.0.0", note = ...)]` (see the projection pair in `datafusion/proto/src/physical_plan/mod.rs` for the exact shape). Deleting the old encode `downcast_ref` arm and repointing the decode arm in the same PR is still required — that's the proof the hook is the live path. - The two foundation issues (#23497 DataSource, #23498 DataSink) gate the scan/sink family items (#23516–#23519); everything under "`datafusion-physical-plan` plans" is independent and can start immediately. - Please comment on the sub-issue you're taking so we don't duplicate work. -- 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]
