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

[jOOQ/jOOQ#7841] DSL.asterisk() does not work well with plain SQL

Browse Source
This commit is contained in:
Lukas Eder 2022-11-25 12:58:25 +01:00
parent e21294cd29
commit 49c9f50d12
1 changed files with 11 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
jOOQ/src/main/java/org/jooq/impl/SelectQueryImpl.java
View File

@ -3730,7 +3730,8 @@ final class SelectQueryImpl<R extends Record> extends AbstractResultQuery<R> imp
// [#7921] TODO Find a better, more efficient way to resolve asterisks
SelectFieldList<SelectFieldOrAsterisk> list = getSelectResolveImplicitAsterisks();
for (SelectFieldOrAsterisk s : list)
int size = 0;
for (SelectFieldOrAsterisk s : list) {
if (s instanceof Field<?> f)
result.add(getResolveProjection(ctx, f));
else if (s instanceof QualifiedAsteriskImpl q) {
@ -3762,6 +3763,15 @@ final class SelectQueryImpl<R extends Record> extends AbstractResultQuery<R> imp
else
throw new AssertionError("Type not supported: " + s);
// [#7841] Each iteration must contribute new fields to the result.
// Otherwise, we couldn't resolve an asterisk, and must fall
// back to determining fields from the ResultSetMetaData
if (size == result.size())
return new SelectFieldList<>();
else
size = result.size();
}
return result;
}