github/jooq
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[jOOQ/jOOQ#10407] Add support for FIREBIRD 4 RETURNING * syntax

Browse Source
This commit is contained in:
Lukas Eder 2022-03-21 14:03:09 +01:00
parent af4c3396ea
commit a1a4caf59f
1 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
jOOQ/src/main/java/org/jooq/impl/AbstractDMLQuery.java
View File

@ -48,6 +48,7 @@ import static java.lang.Boolean.TRUE;
import static org.jooq.SQLDialect.DERBY;
// ...
import static org.jooq.SQLDialect.FIREBIRD;
// ...
import static org.jooq.SQLDialect.H2;
// ...
// ...
@ -183,6 +184,7 @@ abstract class AbstractDMLQuery<R extends Record> extends AbstractRowCountQuery
private static final Set<SQLDialect> NATIVE_SUPPORT_DELETE_RETURNING = SQLDialect.supportedBy(FIREBIRD, MARIADB, POSTGRES, YUGABYTEDB);
private static final Set<SQLDialect> NATIVE_SUPPORT_DATA_CHANGE_DELTA_TABLE = SQLDialect.supportedBy(H2);
private static final Set<SQLDialect> NO_SUPPORT_FETCHING_KEYS = SQLDialect.supportedBy(IGNITE);
private static final Set<SQLDialect> NO_SUPPORT_RETURNING_ASTERISK = SQLDialect.supportedUntil(MARIADB);
@ -829,9 +831,9 @@ abstract class AbstractDMLQuery<R extends Record> extends AbstractRowCountQuery
.sql(' ')
.declareFields(true, c -> c.visit(
// Firebird doesn't support asterisks at all here
// Firebird didn't support asterisks at all here until version 4.0
// MariaDB doesn't support qualified asterisks: https://jira.mariadb.org/browse/MDEV-23178
c.family() == FIREBIRD || c.family() == MARIADB
NO_SUPPORT_RETURNING_ASTERISK.contains(c.dialect())
? new SelectFieldList<>(returningResolvedAsterisks)
: returning
));