Skip to content

Proto: migrate CrossJoinExec + NestedLoopJoinExec #23506

Description

@adriangb

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions