Skip to content

#314: Make LISTAGG overflow Behavior immutable and null-safe#328

Merged
kaklakariada merged 4 commits into
mainfrom
kaklakariada/issue314
Jun 2, 2026
Merged

#314: Make LISTAGG overflow Behavior immutable and null-safe#328
kaklakariada merged 4 commits into
mainfrom
kaklakariada/issue314

Conversation

@kaklakariada

Copy link
Copy Markdown
Contributor

Closes #314

@kaklakariada kaklakariada enabled auto-merge (squash) June 1, 2026 11:53
Comment on lines +736 to +752
final String sqlAsJson = "{" //
+ " \"type\": \"function_aggregate_listagg\"," //
+ " \"name\": \"LISTAGG\"," //
+ " \"distinct\": false," //
+ " \"arguments\": [" //
+ " {" //
+ " \"type\": \"column\"," //
+ " \"columnNr\": 1," //
+ " \"name\": \"USER_ID\"," //
+ " \"tableName\": \"CLICKS\"" //
+ " }" //
+ " ]," //
+ " \"overflowBehavior\":" //
+ " {" //
+ " \"type\": \"ERROR\"" //
+ " }" //
+ "}";

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

painful reminder, still on Java 11 😢

@sonarqubecloud

sonarqubecloud Bot commented Jun 2, 2026

Copy link
Copy Markdown

@kaklakariada kaklakariada merged commit 502935f into main Jun 2, 2026
7 checks passed
@kaklakariada kaklakariada deleted the kaklakariada/issue314 branch June 2, 2026 07:48
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.

Make LISTAGG overflow Behavior immutable and null-safe

2 participants