adriangb opened a new issue, #23506: URL: https://github.com/apache/datafusion/issues/23506
Part of #23494. `joins/cross_join.rs`, `joins/nested_loop_join.rs`. Two children; NLJ adds `JoinType` (by-name match), optional `JoinFilter`, and the `[u32::MAX]` empty-projection sentinel. See `HashJoinExec` in #23496 as the join template. --- Follow the pattern in #23494 (reference impl: #23495 / `ProjectionExec`; full end-state draft: #23496). Add `try_to_proto` inside `impl ExecutionPlan for X`, add the inherent `X::try_from_proto`, repoint the decode arm, and **delete the old central arm in the same PR** (that is the proof the hook is reached). Keep the wire format byte-for-byte identical. -- 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]
