From 2622e3c0f82c4f521aa86a4e1965eeb2ff10e951 Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Wed, 15 Apr 2020 11:13:42 +0200 Subject: [PATCH] [jOOQ/jOOQ#10060] In fact, the behaviour can be seen as correct. The dialects do not agree on whether aggregating empty sets with JSON_ARRAYAGG() leads to NULL or an empty [] --- jOOQ/src/main/java/org/jooq/impl/JSONArrayAgg.java | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/impl/JSONArrayAgg.java b/jOOQ/src/main/java/org/jooq/impl/JSONArrayAgg.java index c13571bfcd..d3392dc925 100644 --- a/jOOQ/src/main/java/org/jooq/impl/JSONArrayAgg.java +++ b/jOOQ/src/main/java/org/jooq/impl/JSONArrayAgg.java @@ -84,11 +84,11 @@ implements JSONArrayAggOrderByStep { // Workaround for https://jira.mariadb.org/browse/MDEV-21914 if (!Tools.isEmpty(withinGroupOrderBy)) - ctx.visit(DSL.concat(inline('['), DSL.coalesce(groupConcat(arguments.get(0)).orderBy(withinGroupOrderBy), inline("")), inline(']'))); + ctx.visit(DSL.concat(inline('['), groupConcat(arguments.get(0)).orderBy(withinGroupOrderBy), inline(']'))); // Workaround for https://jira.mariadb.org/browse/MDEV-21912 else - ctx.visit(DSL.concat(inline('['), DSL.coalesce(groupConcat(arguments.get(0)), inline("")), inline(']'))); + ctx.visit(DSL.concat(inline('['), groupConcat(arguments.get(0)), inline(']'))); break;