Skip to content

[CALCITE-6508] Parse error when using scalar sub-query as operand to Array constructor function#3910

Merged
mihaibudiu merged 1 commit intoapache:mainfrom
ian-bertolacci:CALCITE-6508
Aug 29, 2024
Merged

[CALCITE-6508] Parse error when using scalar sub-query as operand to Array constructor function#3910
mihaibudiu merged 1 commit intoapache:mainfrom
ian-bertolacci:CALCITE-6508

Conversation

@ian-bertolacci
Copy link
Copy Markdown

Allows sub-queries to be used in grammar for operands to square-bracketed array constructor function.

@mihaibudiu mihaibudiu added the LGTM-will-merge-soon Overall PR looks OK. Only minor things left. label Aug 9, 2024
@mihaibudiu mihaibudiu changed the title [CALCITE-6508] Parse error when using scalar sub-query as operant to Array constructor function [CALCITE-6508] Parse error when using scalar sub-query as operand to Array constructor function Aug 9, 2024
@mihaibudiu
Copy link
Copy Markdown
Contributor

I have changed "operant" to "operand" in your JIRA title and PR title, but you will have to change the commit message to match when you squash the commits.

@mihaibudiu
Copy link
Copy Markdown
Contributor

Can you please squash the commits so we can merge this?

@ian-bertolacci
Copy link
Copy Markdown
Author

Done.

@sonarqubecloud
Copy link
Copy Markdown

@ian-bertolacci
Copy link
Copy Markdown
Author

Do I need to trigger the merge? I don't think I have commit permissions.

@mihaibudiu
Copy link
Copy Markdown
Contributor

I did. Didn't notice that's needed.

@mihaibudiu mihaibudiu merged commit 5c0127c into apache:main Aug 29, 2024
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