Skip to content

[CALCITE-7543] RelBuilder.join should preserve variablesSet for RIGHT/FULL joins#4960

Open
iwanttobepowerful wants to merge 1 commit into
apache:mainfrom
iwanttobepowerful:CALCITE-7543
Open

[CALCITE-7543] RelBuilder.join should preserve variablesSet for RIGHT/FULL joins#4960
iwanttobepowerful wants to merge 1 commit into
apache:mainfrom
iwanttobepowerful:CALCITE-7543

Conversation

@iwanttobepowerful
Copy link
Copy Markdown
Contributor

Jira Link

CALCITE-7543

Changes Proposed

Comment thread core/src/test/resources/sql/sub-query.iq Outdated
Copy link
Copy Markdown
Contributor

@mihaibudiu mihaibudiu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fine, but @julianhyde left a comment on the issue, so I will wait for you to reply to that before I approve.

@iwanttobepowerful iwanttobepowerful added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label May 24, 2026
@sonarqubecloud
Copy link
Copy Markdown

@iwanttobepowerful iwanttobepowerful changed the title [CALCITE-7543] RelFieldTrimmer should preserve Join nodes instead of rebuilding them through RelBuilder.join [CALCITE-7543] RelBuilder.join should preserve variablesSet for RIGHT/FULL joins May 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

LGTM-will-merge-soon Overall PR looks OK. Only minor things left.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants