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

[jOOQ/jOOQ#15316] Historic RETURNING implementation only for INSERT

Browse Source 
The historic SQLite RETURNING implementation only works for INSERT statements, running a follow-up query with _rowid_ = last_insert_rowid() predicate.
This commit is contained in:
Lukas Eder 2023-11-24 11:29:31 +01:00
parent f0145338fc
commit 789462bfca
1 changed files with 5 additions and 1 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

@ -938,7 +938,11 @@ abstract class AbstractDMLQuery<R extends Record> extends AbstractRowCountQuery
}
final boolean nativeSupportReturning(Scope ctx) {
return !(ctx.family() == SQLITE && fetchTriggerValuesAfterReturning(ctx))
// [#15316] The historic RETURNING emulation of SQLite was available for INSERT only, using
// _rowid_ = last_insert_rowid() checks, which obviously doesn't work for UPDATE and DELETE
return !(ctx.family() == SQLITE && this instanceof Insert && fetchTriggerValuesAfterReturning(ctx))
&& (this instanceof Insert && !NO_NATIVE_SUPPORT_INSERT_RETURNING.contains(ctx.dialect())
|| this instanceof Update && !NO_NATIVE_SUPPORT_UPDATE_RETURNING.contains(ctx.dialect())
|| this instanceof Delete && !NO_NATIVE_SUPPORT_DELETE_RETURNING.contains(ctx.dialect()));