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

[jOOQ/jOOQ#18931] Avoid DefaultDSLContext allocation in AbstractQueryPart::equals

Browse Source
This commit is contained in:
Lukas Eder 2025-08-27 14:52:38 +02:00
parent 007a62a5ab
commit 0552eb0320
1 changed files with 6 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
jOOQ/src/main/java/org/jooq/impl/AbstractQueryPart.java
View File

@ -165,8 +165,12 @@ abstract class AbstractQueryPart implements QueryPartInternal {
if (that instanceof QueryPart q) {
// [#10635] The two QueryParts may have different Settings attached.
DSLContext dsl1 = Tools.configuration(configuration()).dsl();
DSLContext dsl2 = that instanceof AbstractQueryPart a ? Tools.configuration(a.configuration()).dsl() : dsl1;
Configuration c1 = Tools.configuration(configuration());
Configuration c2 = that instanceof AbstractQueryPart a ? Tools.configuration(a.configuration()) : c1;
// [#18931] Avoid unnecessary DSLContext allocations (e.g. in hot loops)
DSLContext dsl1 = c1.dsl();
DSLContext dsl2 = c2 == c1 ? dsl1 : c2.dsl();
String sql1 = dsl1.renderInlined(this);
String sql2 = dsl2.renderInlined(q);