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

[jOOQ/jOOQ#18474] Race condition in TableImpl initialisation leads to wrong INFO log message about duplicate identities in AbstractTable#getIdentity

Browse Source
This commit is contained in:
Lukas Eder 2025-05-16 08:13:01 +02:00
parent 703f733f79
commit ee30f085d1
1 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

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

@ -505,15 +505,19 @@ implements
@Override
public Identity<R, ?> getIdentity() {
if (identity == null) {
Identity<R, ?> i = null;
for (Field<?> f : fields())
if (f instanceof TableField && f.getDataType().identity())
if (identity == null)
identity = new IdentityImpl(this, (TableField) f);
if (i == null)
i = new IdentityImpl(this, (TableField) f);
else
log.info("Multiple identities", "There are multiple identity fields in table " + this + ", which is not supported by jOOQ");
if (identity == null)
identity = (Identity<R, ?>) IdentityImpl.NULL;
if (i == null)
i = (Identity<R, ?>) IdentityImpl.NULL;
identity = i;
}
return identity == IdentityImpl.NULL ? null : identity;