From 1c367c0511bf1345520fca7cad00616a4e310c65 Mon Sep 17 00:00:00 2001 From: Lukas Eder Date: Wed, 12 Apr 2023 14:09:47 +0200 Subject: [PATCH] [jOOQ/jOOQ#11485] Support for Trino DB - Proper DECIMAL cast in RATIO_TO_REPORT - DOUBLE cast in linear regression functions --- .../jooq/impl/AbstractAggregateFunction.java | 1 + .../java/org/jooq/impl/RatioToReport.java | 22 +++++++++++-------- 2 files changed, 14 insertions(+), 9 deletions(-) diff --git a/jOOQ/src/main/java/org/jooq/impl/AbstractAggregateFunction.java b/jOOQ/src/main/java/org/jooq/impl/AbstractAggregateFunction.java index 33bf8b32d7..2af5f3b31b 100644 --- a/jOOQ/src/main/java/org/jooq/impl/AbstractAggregateFunction.java +++ b/jOOQ/src/main/java/org/jooq/impl/AbstractAggregateFunction.java @@ -555,6 +555,7 @@ implements case FIREBIRD: case HSQLDB: case SQLITE: + case TRINO: return DOUBLE; default: diff --git a/jOOQ/src/main/java/org/jooq/impl/RatioToReport.java b/jOOQ/src/main/java/org/jooq/impl/RatioToReport.java index 758beacf24..e0deedd0a4 100644 --- a/jOOQ/src/main/java/org/jooq/impl/RatioToReport.java +++ b/jOOQ/src/main/java/org/jooq/impl/RatioToReport.java @@ -88,12 +88,6 @@ implements - - - - - - case CUBRID: case FIREBIRD: @@ -113,9 +107,19 @@ implements DataType cast; switch (ctx.family()) { - case SQLITE: cast = DOUBLE; break; - case FIREBIRD: cast = DECIMAL(38, 19); break; - default: cast = DECIMAL; break; + case SQLITE: + cast = DOUBLE; + break; + + + case FIREBIRD: + case TRINO: + cast = DECIMAL(38, 19); + break; + + default: + cast = DECIMAL; + break; } ctx.visit(castIfNeeded(field, cast))