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

[jOOQ/jOOQ#16262] Avoid allocating JoinNode in scope registration of

Browse Source 
tables if unnecessary
This commit is contained in:
Lukas Eder 2024-02-12 11:48:01 +01:00
parent 6f0f180149
commit 2dbcd8a8f4
1 changed files with 13 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

24
jOOQ/src/main/java/org/jooq/impl/AbstractContext.java
View File

@ -1347,20 +1347,22 @@ abstract class AbstractContext<C extends Context<C>> extends AbstractScope imple
Table<?> root,
List<TableImpl<?>> tables
) {
if (result == null)
result = new JoinNode(ctx, root);
if (!tables.isEmpty()) {
if (result == null)
result = new JoinNode(ctx, root);
JoinNode node = result;
for (int i = tables.size() - 1; i >= 0; i--) {
TableImpl<?> t = tables.get(i);
JoinNode node = result;
for (int i = tables.size() - 1; i >= 0; i--) {
TableImpl<?> t = tables.get(i);
if (t.childPath != null)
node = node.pathsToOne.computeIfAbsent(t.childPath, k -> new JoinNode(ctx, t));
else
node = node.pathsToMany.computeIfAbsent(t.parentPath, k -> new JoinNode(ctx, t));
if (t.childPath != null)
node = node.pathsToOne.computeIfAbsent(t.childPath, k -> new JoinNode(ctx, t));
else
node = node.pathsToMany.computeIfAbsent(t.parentPath, k -> new JoinNode(ctx, t));
if (i == 0)
node.references++;
if (i == 0)
node.references++;
}
}
return result;