Skip to content

Isthmus: support LateralJoin conversion (Substrait v0.96.0) #1029

Description

@nielspardon

Substrait v0.96.0 added the LateralJoinRel relation, now modelled in core as io.substrait.relation.LateralJoin. Isthmus does not yet convert it:

  • SqlKindFromRel.visit(LateralJoin) returns SqlKind.JOIN as a placeholder.
  • SubstraitRelNodeConverter (Substrait → Calcite) has no LateralJoin handling and falls back to visitFallback (throws), so a plan containing a lateral join cannot be converted to Calcite.
  • There is no Calcite → Substrait path that produces a LateralJoin (e.g. from a Calcite Correlate / LATERAL / CROSS APPLY).

Follow-up to the core LateralJoin support. Requires proper correlated-reference handling (see the outer-reference typing follow-up).

🤖 Generated with AI

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions