Skip to content

[CALCITE-3651] NPE when convert relational algebra that correlates TableFunctionScan #1711

Merged
hsyuan merged 1 commit into
apache:masterfrom
yanlin-Lynn:CALCITE-3651
Jan 3, 2020
Merged

[CALCITE-3651] NPE when convert relational algebra that correlates TableFunctionScan #1711
hsyuan merged 1 commit into
apache:masterfrom
yanlin-Lynn:CALCITE-3651

Conversation

@yanlin-Lynn

Copy link
Copy Markdown
Contributor

See full stack trace in jiraCALCITE-3651

The reason for NPE coms from the null value for neededAlias attribute when create Result object for TableFunctionScan.

@vlsi

vlsi commented Dec 31, 2019

Copy link
Copy Markdown
Contributor

LGTM

@chunweilei chunweilei left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

+1

@chunweilei chunweilei added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Jan 2, 2020
@hsyuan
hsyuan merged commit 3dd6d85 into apache:master Jan 3, 2020
@yanlin-Lynn

Copy link
Copy Markdown
Contributor Author

Thanks for review and merge.

@yanlin-Lynn
yanlin-Lynn deleted the CALCITE-3651 branch January 3, 2020 09:53
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.

4 participants