Skip to content

[CALCITE-5390] RelDecorrelator throws NullPointerException#4776

Merged
iwanttobepowerful merged 1 commit intoapache:mainfrom
iwanttobepowerful:calcite5390
Feb 24, 2026
Merged

[CALCITE-5390] RelDecorrelator throws NullPointerException#4776
iwanttobepowerful merged 1 commit intoapache:mainfrom
iwanttobepowerful:calcite5390

Conversation

@iwanttobepowerful
Copy link
Contributor

@iwanttobepowerful iwanttobepowerful commented Jan 28, 2026

@iwanttobepowerful
Copy link
Contributor Author

I think the frameStack introduced by @suibianwanwank simulates the de Bruijn indexes mentioned by @julianhyde .

return map.get(r);
}

private RelNode getCorRel(CorRef corVar) {
Copy link
Contributor Author

Choose a reason for hiding this comment

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

We might be able to delete getCorRel. I found during debugging that this function is never reached anymore.

@iwanttobepowerful iwanttobepowerful force-pushed the calcite5390 branch 2 times, most recently from 8e7ac0c to 3bab8c4 Compare January 29, 2026 08:44
@sonarqubecloud
Copy link

@iwanttobepowerful
Copy link
Contributor Author

@dssysolyatin @rubenada @mihaibudiu @NobiGo @suibianwanwank
can you help review this PR? Thx.

@iwanttobepowerful iwanttobepowerful added LGTM-will-merge-soon Overall PR looks OK. Only minor things left. labels Feb 9, 2026
@mihaibudiu
Copy link
Contributor

@iwanttobepowerful I think you have permissions to merge this PR, so I will let you do it.
Do not forget to resolve the corresponding JIRA issue, and to add the commit to the issue when you do that.

@iwanttobepowerful iwanttobepowerful merged commit 69a7a06 into apache:main Feb 24, 2026
49 of 50 checks passed
@iwanttobepowerful
Copy link
Contributor Author

@iwanttobepowerful I think you have permissions to merge this PR, so I will let you do it. Do not forget to resolve the corresponding JIRA issue, and to add the commit to the issue when you do that.

I don't have access to update the Jira ticket status.

My jira ID is zwh.

@xiedeyantu
Copy link
Member

@iwanttobepowerful I think you have permissions to merge this PR, so I will let you do it. Do not forget to resolve the corresponding JIRA issue, and to add the commit to the issue when you do that.

I don't have access to update the Jira ticket status.

My jira ID is zwh.

You can email private@calcite.apache.org; a PMC member should be able to help you resolve this. I probably don't have the same permissions as you to handle this issue.

@iwanttobepowerful iwanttobepowerful deleted the calcite5390 branch February 24, 2026 07:55
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