Skip to content

fix(sql): expression alias for array don't use the correct syntax#5922

Merged
bluestreak01 merged 3 commits intomasterfrom
raph_expr_alias_array
Jul 10, 2025
Merged

fix(sql): expression alias for array don't use the correct syntax#5922
bluestreak01 merged 3 commits intomasterfrom
raph_expr_alias_array

Conversation

@RaphDal
Copy link
Copy Markdown
Contributor

@RaphDal RaphDal commented Jul 9, 2025

Generated aliases from expression should be usable directly in an SQL query. This PR does that for array so that when a user creates an array using the array function, it's now aliases to array[] instead of array().

@RaphDal RaphDal changed the title fix(sql): Expression alias for array don't use the correct syntax fix(sql): expression alias for array don't use the correct syntax Jul 9, 2025
@glasstiger
Copy link
Copy Markdown
Contributor

[PR Coverage check]

😍 pass : 11 / 11 (100.00%)

file detail

path covered line new line coverage
🔵 io/questdb/griffin/model/ExpressionNode.java 11 11 100.00%

@bluestreak01 bluestreak01 merged commit a2ff0b2 into master Jul 10, 2025
34 checks passed
@bluestreak01 bluestreak01 deleted the raph_expr_alias_array branch July 10, 2025 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants