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

[jOOQ/jOOQ#17425] DSLContext::fetchCount for selects with nested records causes DataAccessException

Browse Source
This commit is contained in:
Lukas Eder 2024-10-14 14:03:42 +02:00
parent d0164385c9
commit bcb49ec0e5
2 changed files with 13 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
jOOQ/src/main/java/org/jooq/impl/AliasedSelect.java
View File

@ -42,6 +42,7 @@ import static org.jooq.impl.DSL.selectFrom;
import static org.jooq.impl.Names.NQ_SELECT;
import static org.jooq.impl.Names.N_T;
import static org.jooq.impl.SubqueryCharacteristics.DERIVED_TABLE;
import static org.jooq.impl.Tools.anyMatch;
import static org.jooq.impl.Tools.collect;
import static org.jooq.impl.Tools.fieldNames;
import static org.jooq.impl.Tools.flattenCollection;

13
jOOQ/src/main/java/org/jooq/impl/FetchCount.java
View File

@ -40,6 +40,7 @@ package org.jooq.impl;
import static org.jooq.impl.DSL.count;
import static org.jooq.impl.DSL.select;
import static org.jooq.impl.Tools.anyMatch;
import org.jooq.Configuration;
import org.jooq.Context;
@ -66,7 +67,17 @@ final class FetchCount extends AbstractResultQuery<Record1<Integer>> implements
@Override
public final void accept(Context<?> ctx) {
ctx.visit(select(count).from(new AliasedSelect<>(query, true, true, false).as("t")));
// [#17425] Without native support for nested records, the projected aliases will be off
// There's likely a more generic bug, related to AliasedSelect in general, not just to
// FetchCount, where this fix should be moved, instead
AliasedSelect<?> s = new AliasedSelect<>(query, true, true, false);
ctx.visit(select(count).from(
RowAsField.NO_NATIVE_SUPPORT.contains(ctx.dialect()) && anyMatch(query.getSelect(), f -> f.getDataType().isRecord())
? s.as("t", Tools.EMPTY_STRING)
: s.as("t")
));
}
@Override