apache / apache/beam

BeamSideInputJoinRel should swap nodes in the Rule, not in expand

Open
#20,353 0 comments 0 reactions 0 assignees View on GitHub
dsl improvement P3 sql
Dominant language
Java
Stars
8.7k
Forks
4.7k
Avg merge
1d 20h
Merged PRs (30d)
196

Description

BeamSideInputJoinRel conditionally swaps the order of inputs in expand. This makes the code very difficult to debug from customer logs. We should do the swapping in the Rule instead so the physical implementation logs actually reflect the physical implementation.

Imported from Jira [BEAM-10477](https://issues.apache.org/jira/browse/BEAM-10477). Original Jira may contain additional context.
Reported by: apilloud.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.