Implements Jakarta Persistence 3.2 by cristof · Pull Request #144 · apache/openjpa

cristof · 2026-05-16T18:41:02Z

Hi! This work is an effort to implement JPA 3.2. I've started it a long time ago. Richard Zowalla picked it up and, with AI help (Claude), implemented the missing features, including some JPA <3.0 that weren't implemented.
I've tested it against default database, h2, mariadb(lts) e postgresql (18).
Please, check it against your favorite DB so we may fix some edge cases. It would be great if you can run TCK to be sure of the implementations.

…ptional in JPQL" implementation

…fixes

…Year a

…public and serializable

…eady working)

…NativeQuery

…DL generation

…nd named result set mappings to generated StaticMetamodel

… list of defined annotations on StaticMetamodel

…ields of Java enums

… have Property access only Getter/Setters according to Java Bean Style should work.

…ed when a resource-local transaction is active, roll it back before closing. For managed transactions, defer closing until the transaction completes.

…OpenJPA created schema

…s unversioned types with a clear message

…ner are improved

…-2940

solomax · 2026-08-18T08:42:39Z

All,

There are lot's of unaddressed comments, but I'm afraid working on them in this PR is a pain :(

Maybe it worth to merge this one and address remaining in a separate PR?

Shall this one be squashed WDYT?

rzo1 · 2026-08-18T08:44:54Z

All,

There are lot's of unaddressed comments, but I'm afraid working on them in this PR is a pain :(

Maybe it worth to merge this one and address remaining in a separate PR?

Shall this one be squashed WDYT?

I think people opted for not squashing to remain history. I am fine with merging, but we need a way to track the open stuff to avoid loosing it on the long run.

Perhaps it would make sense to create JIRAs for the open things we need to fix ?

(or we use AI to create a table of unresolved comments to aggregate them and than automtically create the JIRAs, wdyt?)

solomax · 2026-08-18T08:52:40Z

@rmannibucau could you please mark all answered comments which doesn't require any further work as resolved?

I can then try to move all remaining issues to the JIRA (hopefully I can do it with the links to the particular comment)

rzo1 · 2026-08-18T08:55:59Z

Updated: 54 unresolved review threads, plus the MySQL TCK comment @solomax asked to include (last row). The EnumValueHandler @EnumeratedValue thread is now resolved and has dropped off.

Status = whether @rmannibucau's comment has been answered in-thread yet — Open means nobody has replied, so it still needs a decision before it can become a JIRA.

ID	Link	Title	Comment	Author	Status
3426807128	3426807128	Null state manager cases should throw instead of skip	wonder if these null cases shouldn't throw, means enhancement is broken or setup (agent) is broken no?	rmannibucau	Discussed · 1 reply, last @rzo1
3682999074	3682999074	New JPQL keywords no longer usable as aliases	(medium) `identification_variable()` only accepts `<IDENTIFIER>`, so every newly introduced token (ID, VERSION, RIGHT, ON, NULLS, FIRST, LAST, CAST, STRING, UNION, INTERSECT, EXCEPT, TREAT, ...) can no longer be used as an identification variable or result alias. Existing queries like `SELECT e.id AS id ... ORDER BY id` or aliases named `first`/`on` now fail to parse - a backward-compat regression worth documenting or mitigating with soft keywords. (re line 1560, outside the diff hunks)	rmannibucau	Discussed · 1 reply, last @cristof
3682999176	3682999176	ID()/VERSION() limited to equality against parameters	(medium) `entity_id_or_version_comp()` only allows `ID(x)`/`VERSION(x)` compared with `=`/`<>` against an input parameter. Spec-legal forms like `ID(e) = 5`, `VERSION(e) >= :v` or `ID(a) = ID(b)` do not parse - intentional first step, or should these functions be reachable from the general comparison/arithmetic productions?	rmannibucau	Discussed · 1 reply, last @cristof
3682999345	3682999345	Broadened auto-flush before every query hurts perf	(medium) With FLUSH_TRUE (default FlushModeType.AUTO) this now flushes before every query as soon as anything in the context is dirty, and the `flush == FLUSH_TRUE` addition at line 1033 also overrides the IgnoreChanges setting. That defeats the access-path optimization and can be a real perf regression in write-heavy transactions - can the broadened check be limited to types related to the query access path, or guarded by a compatibility flag?	rmannibucau	Open · no reply
3682999520	3682999520	Full context flush after in-memory bulk update/delete	(medium) The new `_broker.flush()` after in-memory bulk delete (and update at line 1156) flushes the entire persistence context as a side effect of `executeUpdate()`, changing when unrelated pending changes hit the database (locks, triggers, constraint timing). Is flushing only the affected instances feasible, or is the full flush deliberate?	rmannibucau	Open · no reply
3682999635	3682999635	Transient object wrongly assumed detached in flush check	(medium) This early-return treats any unmanageable instance of a known entity type as detached without verifying it was ever persisted, so a truly transient unenhanced object referenced without cascade now passes the check and fails later (or silently persists a broken FK). Line 824 also adds `_broker.isDetached(obj, true)`, a DB round-trip inside the flush path for every manageable object without a state manager. Can both paths at least verify a non-null/assigned identity before assuming detached?	rmannibucau	Open · no reply
3682999733	3682999733	Embedded converter override leaks to shared embeddable meta	(medium) `propagateEmbeddedConverters()` sets the converter on the shared embeddable ClassMetaData. If two entities embed the same embeddable but only one declares `@Convert(attributeName=...)`, the override leaks to all other usages (last-resolved wins). Should the override only apply to the per-embedding `embeddedMeta` copy?	rmannibucau	Open · no reply
3682999838	3682999838	Unsafe publication of cached AttributeConverter instance	(medium) The converter instance is now created once and cached in a non-volatile field without synchronization; FieldMetaData is shared across brokers/threads, so this is unsafe publication and a behavior change (previously a new converter instance per conversion, now one shared instance that must be thread-safe). Consider thread-safe caching and documenting that AttributeConverters are treated as shared/stateless.	rmannibucau	Open · no reply
3683000132	3683000132	extractSchemaGenObjects mutates caller's properties map	(medium) `extractSchemaGenObjects` calls `map.remove(key)` on the caller-supplied properties map - this mutates user configuration and throws UnsupportedOperationException for unmodifiable maps (e.g. `Map.of(...)` passed to `createEntityManagerFactory`). Can the Writer/Reader values be captured without mutating the input?	rmannibucau	Open · no reply
3683000248	3683000248	In-memory ID(), set ops and nullPrecedence unimplemented	(medium) `getNativeObjectId` returns the same `GetObjectId` as `getObjectId`, which evaluates to the internal ObjectId wrapper (e.g. LongId), not the raw PK value - so an in-memory `ID(e) = :id` comparison against the plain key may never match; should it unwrap like the JDBC side? Similarly `setOperands`/`setOperationType` and `nullPrecedence` appear consumed only by the JDBC store, so in-memory execution of UNION/INTERSECT/EXCEPT or NULLS FIRST/LAST silently produces wrong results - should the in-memory path reject or implement them?	rmannibucau	Open · no reply
3683000358	3683000358	hasEmbeddableAnnotation relies on annotation simple name	(low) `hasEmbeddableAnnotation` matches any annotation whose simple name is "Embeddable" (from any package) and misses embeddables declared only in orm.xml - would checking the repository metadata (e.g. `fmd.getEmbeddedMetaData()`) be more reliable than reflection on annotation names?	rmannibucau	Open · no reply
3683000504	3683000504	Direct access to ImplHelper._unenhancedInstanceMap	(low) Reaching into `ImplHelper._unenhancedInstanceMap` (a public mutable static field) directly is fragile - suggest a small `ImplHelper.registerUnenhancedInstance(obj, pc)` method instead of exposing the raw map.	rmannibucau	Open · no reply
3683001426	3683001426	dropExcludedTypeTables drops user tables with raw SQL	(high) `dropExcludedTypeTables` issues a raw `"DROP TABLE " + tableName` bypassing the dictionary (no identifier quoting/`toDBName`, no CASCADE handling, will fail on Postgres with dependent constraints) and swallows every exception at trace level. More fundamentally, dropping a user table because its type was excluded from synchronization is destructive - an excluded type may be a table managed externally on purpose. Is this only here to make a specific test pass? It probably should not ship in production code.	rmannibucau	Open · no reply
3683001538	3683001538	NULLS FIRST/LAST string rewrite breaks on commas, triplicated	(high) The NULLS FIRST/LAST emulation finds the last order term with `sql.lastIndexOf(", ", termDirStart)`, so any ORDER BY expression containing a comma (e.g. `COALESCE(t0.x, 0) DESC`) is split mid-argument-list and the rewrite produces corrupt SQL; duplicating the expression also duplicates `?` markers without duplicating bound parameters. The block is triplicated in MariaDBDictionary (~575) and SQLServerDictionary (~484) - can it be extracted to a shared helper operating on the order term before it is appended to the buffer instead of string-parsing it back out?	rmannibucau	Discussed · 3 replies, last @solomax
3683001854	3683001854	TREAT discriminator filter excludes subclasses of target	(medium) `appendTreatDiscriminator` emits `disc = <treated class value>` only. Per JPA, `TREAT(x AS Middle)` must also match subtypes of Middle, so for a 3-level hierarchy this incorrectly filters out instances of Middle's subclasses - it should be an IN over the discriminator values of the treated class and all mapped subclasses. It also only handles `cols[0]` of the discriminator.	rmannibucau	Open · no reply
3683001994	3683001994	Schema-gen methods leak a Broker just to get classloader	(medium) `createPersistenceStructure`, `dropPersistenceStrucuture`, `validatePersistenceStruture` and `truncateData` each create a Broker via `super.newBrokerImpl(...)` that is only used for `getClassLoader()` and never closed - a broker (and its resources) leaked per call. Can the classloader be obtained without instantiating a broker?	rmannibucau	Open · no reply
3683002126	3683002126	Static _droppedTables shared across concurrent EMFs	(medium) `_droppedTables` is JVM-global static mutable state and `clearDroppedTables()` is invoked from JDBCBrokerFactory whenever any EMF with schema-gen properties spins up - with two persistence units initializing concurrently (common in app servers) one EMF wipes the other's in-flight tracking. Matching also uses `toUpperCase()` without `Locale.ROOT` (here, line 1239 and 1519-1529) and compares a full identifier against a name regex-stripped from raw DDL, so schema-qualified or quoted names will not match. Could this state live on the configuration instead?	rmannibucau	Open · no reply
3683002267	3683002267	Range ignored in executeSetOperatorQuery	(medium) `executeSetOperatorQuery` receives `range` but never uses it, so `setFirstResult`/`setMaxResults` on a UNION/INTERSECT/EXCEPT query are silently ignored. Should the range at least be applied via RangeResultObjectProvider like the normal path does?	rmannibucau	Open · no reply
3683002420	3683002420	Bulk delete no longer cleans dependent/collection rows	(medium) Removing the `getCascadeDelete() != CASCADE_NONE -> INVALID` guard means bulk DELETE no longer falls back to loading instances for entities with cascading/dependent fields. JPA 4.10 (no cascade on bulk delete) is fine, but this strategy also handled dependent-field cleanup: join-table / element-collection rows previously removed via the in-memory fallback can now be left orphaned unless the DB has ON DELETE CASCADE. Was that trade-off verified for the element-collection case?	rmannibucau	Open · no reply
3683002577	3683002577	Long cast uses decimalTypeName instead of bigint	(medium) Casting to Long uses `dict.decimalTypeName`; on MySQL `CAST(x AS DECIMAL)` defaults to DECIMAL(10,0), so large long values overflow/truncate - why not `bigintTypeName` for the long case? Also `getDbNumberTargetTypeName` sanitizes the `{0}` size suffix while the sibling TypecastAsString.java:152 appends `dict.varcharTypeName` raw - the two siblings should share the same sanitize logic.	rmannibucau	Open · no reply
3683002730	3683002730	VersionVal NPEs on surrogate or missing version	(medium) `getColumns()` NPEs when the entity has a surrogate version or no version at all (`getVersionFieldMapping()` returns null), so `VERSION(e)` on such an entity dies with NullPointerException instead of a meaningful error - `initialize` should validate this like it validates the class mapping. The error at line 99 also reuses the `bad-getobjectid` message, which is misleading for a VERSION() failure.	rmannibucau	Open · no reply
3683003006	3683003006	IdClass paths swallow exceptions and rely on field order	(medium) The IdClass reconstruction/extraction paths `catch (Exception)` and silently continue (here the field stays null; in `toDataStoreValue` at line 227 the PK columns get nulls written). Swallowing exceptions around primary-key values risks silently persisting/loading corrupt identities - at minimum a warn log, arguably a StoreException. Also `getInstanceFields` (line 448) maps IdClass fields to columns by `getDeclaredFields()` order, which the JVM does not guarantee - matching by name would be safer.	rmannibucau	Open · no reply
3683003132	3683003132	Instant.now() precision loss on DB round-trip	(medium) `Instant.now()` carries micro/nano precision on modern JVMs; if the version column's precision is lower (MySQL TIMESTAMP defaults, Oracle DATE) the value read back differs from the in-memory version, producing spurious optimistic-lock failures on the next flush - the same reason TimestampVersionStrategy deliberately uses millisecond granularity. Should this truncate to a precision known to survive the DB round-trip?	rmannibucau	Discussed · 1 reply, last @solomax
3683003222	3683003222	Null named parameter falls through to positional lookup	(medium) Using `userParams.get(name) == null` to fall through to positional lookup means an explicitly bound null named parameter is silently replaced by whatever is registered under the position key - `containsKey` should distinguish "bound to null" from "absent". Related: the new `c.getIndex() < params.length` guards at lines 188/195 silently skip binding instead of failing, turning a caller bug into an unbound-parameter SQLException far from the cause.	rmannibucau	Open · no reply
3683003316	3683003316	storeCharsAsNumbers default flip breaks existing schemas	(medium) Flipping `storeCharsAsNumbers` to false for PostgreSQL >= 9 changes the default mapping for existing applications: char fields previously stored in INTEGER columns will now map/validate as CHAR, so schemas created by older OpenJPA versions fail validation or read wrongly after an upgrade. Intended compatibility break? Should be release-noted and overridable.	rmannibucau	Open · no reply
3683003416	3683003416	Ceiling mutates shared operator field in appendTo	(low) `appendTo` mutates the instance field `operator` before delegating to `super.appendTo` (same pattern in NaturalLogarithm.java:49). Compiled query plans are cached and shared, so this is a data race on shared state; passing the dictionary function down locally (e.g. an `appendTo(..., String operator)` overload in UnaryOp) would keep the Val immutable.	rmannibucau	Discussed · 1 reply, last @cristof
3683003654	3683003654	Converter instantiated reflectively, skips CDI and nulls	(low) Two questions: (1) the converter is instantiated via `getDeclaredConstructor().newInstance()`, bypassing CDI-managed converters (JPA allows converters as CDI beans with injection); (2) why reflection/`findMethod` instead of casting to `jakarta.persistence.AttributeConverter` and calling it directly? Note also the `val == null` short-circuits mean a converter mapping null to a default value is never consulted for nulls.	rmannibucau	Open · no reply
3683004301	3683004301	String cache mode properties no longer converted to enum	(medium) The early `return (T) value;` now also fires for String values of `cache.retrieveMode`/`cache.storeMode`, skipping the enum conversion below. Since EntityManagerImpl now exposes `setCacheRetrieveMode(CacheRetrieveMode)`, `em.setProperty("jakarta.persistence.cache.retrieveMode", "USE")` will try to inject a raw String into the enum setter and fail, whereas the old code converted it. Should Strings still be converted to the target enum here?	rmannibucau	Open · no reply
3683004426	3683004426	treat() join overloads silently cast without narrowing	(medium) Only `treat(Root)` gets a real implementation (`RootImpl.TreatedRoot`); all the join overloads (lines 429-450) and `treat(Path)` (line 458) just cast and return the same object. That is a silent no-op: no type narrowing is applied, but instead of the previous explicit UnsupportedOperationException users now get wrong behavior with no diagnostic. Could the unsupported overloads keep throwing (or get a TreatedJoin analogous to TreatedRoot)?	rmannibucau	Open · no reply
3683004555	3683004555	version 3.1 persistence.xml validated against 3.0 XSD	(medium) Documents declaring `version="3.1"` are routed to `persistence_3_0.xsd.rsrc`, but that schema declares `version` as `fixed="3.0" use="required"`, so such documents always fail XSD validation - and with the new SAX-rethrow logic at line 570 this now aborts unit discovery instead of being skipped. Is the branch intentional, and should not it validate against a schema that accepts "3.1"? (Note the copy-pasted `PERSISTENCE_XSD_3_0` in the 3.1 condition.)	rmannibucau	Open · no reply
3683004644	3683004644	convert() mutates caller config, ignores mappingFiles	(medium) `convert(PersistenceConfiguration)` mutates the caller's configuration object (`config.property(...)` for MetaDataFactory and noPersistenceXMLResource) - a surprising side effect for a converter, and repeated `createEntityManagerFactory(config)` calls keep appending. It also ignores `config.mappingFiles()` and `config.qualifiers()` entirely, and `setPersistenceUnitName(config.name())` is called twice (lines 648 and 662). Could the OpenJPA-specific properties go into the returned map instead, and mapping files be wired through?	rmannibucau	Open · no reply
3683004763	3683004763	Typos, stray semicolon, and validate() masking unsupported	(medium) Nits before this freezes: the delegated `BrokerFactory` method names carry the `Strucuture`/`Struture` typos, `truncateData();;` has a double semicolon at line 64, and the `(Exception) ex` cast at line 58 is redundant. Also `validate()` wraps even UnsupportedOperationException from stores that do not implement validation into `SchemaValidationException("Schema could not be validated: null")`, misreporting a missing capability as a validation failure.	rmannibucau	Open · no reply
3683004886	3683004886	getReference(entity) fails on composite ids	(medium) `getReference(T entity)` only extracts the PK when `pkFields.length == 1`; for composite/IdClass/EmbeddedId entities `pk` stays null and the call fails downstream with a misleading "null pk" IllegalArgumentException. Also `entity.getClass()` (line 2657) may be a runtime subclass without direct metadata. Could composite ids be supported via `broker.getObjectId`, or at least fail with an explicit "composite id not supported" message?	rmannibucau	Open · no reply
3683005009	3683005009	addNamedQuery relabels criteria query as JPQL	(medium) `addNamedQuery` re-labels a criteria query as JPQL using `queryImpl.getQueryString()`, which for criteria queries is the CQL/toString rendering - OpenJPA has never guaranteed that string is parseable JPQL (parameter rendering, literals, treated paths). Has this been exercised beyond simple queries? The three `catch (Exception) { // ignore }` blocks below (lines 512-540) would also hide genuine failures - they could be narrowed to IllegalStateException where the spec defines it.	rmannibucau	Open · no reply
3683005121	3683005121	Dead access-type annotation helper methods	(medium) `hasMixedAnnotations`, `hasFieldStrategyAnnotations` (547) and `hasGetterStrategyAnnotations` (556) are never called from anywhere - dead code from an earlier iteration of the access-type rework; suggest removing.	rmannibucau	Open · no reply
3683005250	3683005250	getProperties() caches map missing EM-level defaults	(low) The old else branch seeding `getProperties()` from a throwaway EM was removed, so the result now depends on whether an EM was created before the first call - and since the map is cached, the EM-level defaults are then permanently missing. Intentional, or should the cache be invalidated once `emEmptyPropsProperties` becomes available?	rmannibucau	Open · no reply
3683005413	3683005413	Locale-sensitive toUpperCase on temporal field name	(low) `field.toString().toUpperCase()` is locale-sensitive ("minute" breaks under a Turkish default locale). Prefer `toUpperCase(Locale.ROOT)`, and ideally key off the known LocalDateField/LocalTimeField constants rather than `toString()`.	rmannibucau	Discussed · 1 reply, last @solomax
3683005700	3683005700	Dropped non-Serializable IdClass warning	(low) The "IdClass does not implement Serializable" warning was silently dropped both here and in AnnotationPersistenceMetaDataParser. Deliberate (3.2 relaxes it?) or lost in the rewrite?	rmannibucau	Open · no reply
3683005795	3683005795	Mandatory JPA 3.2 methods throw UnsupportedOperationException	(low) `getNamedQueries(Class)` still throws UnsupportedOperationException, as do `EntityManagerImpl.createQuery(TypedQueryReference)` (EntityManagerImpl.java:2779) and `find(EntityGraph, Object, FindOption...)` (EntityManagerImpl.java:2645). These are mandatory JPA 3.2 API - planned before merge, or tracked in a follow-up JIRA? Worth referencing the issue in the exception message.	rmannibucau	Open · no reply
3683006065	3683006065	setTimeout(null) cannot clear a previously set query timeout	(low) `setTimeout(null)` is silently ignored, so once a timeout is set it can never be cleared through this API (and `getTimeout()` keeps returning the stale value). Should null reset the fetch plan's query timeout to its default?	rmannibucau	Open · no reply
3683006222	3683006222	Externalized-parameter tests reduced to no-op assertions	(high) These three tests kept their names ("CanDetectExternalized...") but no longer detect anything: the `getExpressions()` helper was deleted and the `isUsingExternalizedParameter(...)` assertions were replaced by `assertNotNull(getResultList())`, which can never fail. Was externalized-parameter detection actually removed from the prepared-query cache, or can the original assertions be restored? As written the tests are no-ops.	rmannibucau	Discussed · 1 reply, last @solomax
3683006359	3683006359	LOCAL TIME test query is a tautology matching all rows	(high) The query in testGetCurrentLocalTime was changed to `localTimeField < LOCAL TIME OR localTimeField >= LOCAL TIME`, a tautology matching every row no matter what LOCAL TIME evaluates to. The test now only verifies the query parses. Could we keep an assertion that actually constrains the result (e.g. compare against a value persisted just before)?	rmannibucau	Discussed · 1 reply, last @solomax
3683006506	3683006506	Results always materialized, lazy ResultList behavior lost	(high) Several assertions here (and in TestQueryTimeout) were inverted from "iterator must be invalid after query/EM close" to "iterator still works because results are now an ArrayList snapshot". Which JPA 3.2 clause requires this? More importantly, does this mean the lazy ResultList (openjpa.FetchBatchSize streaming results) is gone and results are always fully materialized? That would be a significant memory/perf behavior change deserving explicit discussion and release-noting, not just adjusted tests.	rmannibucau	Open · no reply
3683006632	3683006632	Bulk delete no longer cleans join-table rows	(high) testSingleDelete/testBulkDelete were inverted from "addresses deleted" to "addresses remain" citing spec 4.10 - but that clause has said bulk delete does not cascade since JPA 1.0, so this is a deliberate break with long-standing OpenJPA behavior rather than something new in 3.2. Also the `assertSQL("DELETE FROM .J_PERSON_ADDRESSES .")` assertions were dropped entirely: are the join-table rows still cleaned up, or do we now leave dangling rows pointing at deleted pks (FK violation on constrained schemas)? Please keep an assertion on the join-table state and consider a compatibility option plus release note. Same change in TestBulkJPQLAndDataCache.java:122.	rmannibucau	Open · no reply
3683006744	3683006744	Deeply nested multiselect extension now rejected	(medium) testDeeplyNestedShape previously verified OpenJPA's documented extension supporting arbitrary nesting of tuple/array selections (the old comment even said the negative test was retired for that reason); it is now inverted to expect IllegalArgumentException. The spec's "must not" has been there since 2.0, so this drops a working extension existing applications may rely on. Intentional, and should it be release-noted?	rmannibucau	Open · no reply
3683006891	3683006891	Non-entity classes in persistence.xml silently skipped	(medium) Inverted from "listing a non-persistent class in persistence.xml raises ArgumentException" to "non-entity classes are silently skipped". Silent skipping also hides real user errors (forgotten annotations, broken enhancement) that previously failed fast. Is this required by a specific TCK test? If so, could we at least keep a warning log and reference the TCK requirement in a comment?	rmannibucau	Open · no reply
3683007048	3683007048	Renamed getaXxx accessors drop JavaBeans naming coverage	(medium) The accessors were renamed from the JavaBeans-Introspector style (getaCAPITAL/getaWord/isaBoolean for fields aCAPITAL/aWord/aBoolean) to getACAPITAL/getAWord/isABoolean. This test existed precisely to cover the former naming, so property-access entities using IDE-generated getaXxx accessors would silently stop being recognized. Did 3.2 change property-name resolution, or is this adapting the test to a regression? Could both variants stay covered?	rmannibucau	Open · no reply
3683007155	3683007155	Map-key column default KEY0 to entityCs_KEY breaks upgrades	(medium) Changing the expected default map-key column from KEY/KEY0 to entityCs_KEY (also TestContainerSpecCompatibilityOptions.java:426 and the KEY0 -> photos_KEY expected SQL in TestTypesafeCriteria) is spec-correct, but silently changes DDL/SQL against existing schemas created by older OpenJPA versions - upgrades will not find the KEY0 column. Should this be gated behind a compatibility option (this test class exists exactly for that) and called out in migration notes?	rmannibucau	Open · no reply
3683007396	3683007396	assertSQL now ignores identifier delimiters suite-wide	(medium) assertSQL() now also matches after stripping identifier delimiters (same idea as the new same() helper in AbstractCriteriaTestCase.java:159). This globally relaxes every SQL assertion in the suite and would mask regressions in delimited-identifier handling. Could the delimiter-insensitive comparison be opt-in, or at least documented why it became necessary?	rmannibucau	Discussed · 1 reply, last @solomax
3683007607	3683007607	testNotLoadedLazy duplicates eager check, lazy path untested	(low) In this rewrite, testNotLoadedLazy still calls `verifyIsLoadedEagerState(false)` (duplicate of testNotLoadedEager), so the lazy not-loaded path (`verifyIsLoadedLazyState(false)`) remains untested. Also createLazyEntity builds a RelEntity that is never persisted, which the new testLoadingLazyAttributeByName relies on.	rmannibucau	Open · no reply
3683007780	3683007780	Deletion of TestSecurityContext needs rationale in PR	(low) This is the only deleted test file; deletion looks justified (SecurityManager removal on modern JDKs), but please state the rationale in the PR description so the removal is clearly intentional.	rmannibucau	Open · no reply
3683007909	3683007909	MariaDB branch accepts either message without naming versions	(low) The MariaDB branch now accepts either the per-row or first-row failed object/message "across versions", weakening the exact-match check still applied to other DBs. If specific MariaDB versions differ, could the comment name the versions observed so this does not quietly absorb future regressions?	rmannibucau	Open · no reply
3683008653	3683008653	Leftover [2024] bracket and stale CDDL sentence vs EFSL 1.1	(medium) Two nits: the template brackets survived in "Copyright (c) [2024] Eclipse Foundation AISBL", and the next paragraph still says "OpenJPA elects to include this software in this distribution under the CDDL license" while the surrounding text was changed to EFSL 1.1 - should the CDDL sentence be updated for consistency?	rmannibucau	Open · no reply
3683008798	3683008798	Oracle profile bind-mount jdbc_oradata outside the checkout	(medium) The oracle profile bind-mounts `${project.basedir}/../jdbc_oradata`, which for the root pom resolves outside the checkout (and to a different path per module since profiles are inherited); run-build-matrix.sh then pre-creates it with `chmod a+rwx` and warns that cleanup needs sudo. Would a named docker volume (or a path under `target/`) be cleaner so nothing leaks outside the repo?	rmannibucau	Discussed · 9 replies, last @solomax
5011202275	5011202275	MySQL TCK: char field insert truncated on CHAR column	It seems some errors in MySQL TCK tests are caused by: [code] While trying to persist entity with: [code] Table is create with: [code] java oblect is created with: [code] Something weird :(((	solomax	Open · no reply

…licate values, guard lazy init

solomax · 2026-08-18T09:42:35Z

@rzo1 could you please add #144 (comment) to the table?

And let's wait for @rmannibucau to resolve at least something from this huge table ....

rzo1 · 2026-08-18T09:49:51Z

@rzo1 could you please add #144 (comment) to the table?

And let's wait for @rmannibucau to resolve at least something from this huge table ....

Updated + Status Column ;-) - so lets wait and update it so we can proceed with the Jiras later on.

…to new container

solomax · 2026-08-19T02:32:28Z

@@ -97,6 +99,37 @@ public void testQueryTimeOutExceptionWhileQueryingWithLocksOnAlreadyLockedEntiti
        }
    }

+    public void testLockTimeOutExceptionWhileQueryingWithLocksOnAlreadyLockedEntitiesOption() {


This test seems to hang for Oracle:8.4.0 is it by design?

What is the rule to set DBDictionary.supportsQueryTimeout ?

I've added TestTimeout == 30min for this test for now, not sure how to properly fix it :((

same story with TestPessimisticLockException :(((
I think migrating to JUnit5 with @Timeout annotations might be good idea :)

I 'd like to, but it's something we should deal after this release, don't you think?

I 'd like to, but it's something we should deal after this release, don't you think?

+1

solomax · 2026-08-20T08:37:22Z

Hello All,

I have created https://issues.apache.org/jira/browse/OPENJPA-2945 and all remaining issues as it's subtasks

@cristof please merge this one :) this honor is yours :)

I hope to get your and @rzo1 help on resolving remaining issues :)

cristof · 2026-08-20T09:13:08Z

Merging, as requested. Sure will help with the issues!

…a Persistence 3.2) Documents the incompatibilities, default changes, platform/dependency requirements, database-specific changes, SPI changes and notable new features introduced by the Jakarta Persistence 3.2 implementation (#144), derived from the merged code changes and the PR review discussion.

cristof and others added 30 commits November 17, 2025 08:55

[WIP][OPENJPA-2940] Implemented ID(obj) FUNCTION on JPQL

bef97d8

[WIP][OPENJPA-2940] Implements version JPQL function

2e9b0cd

[WIP][OPENJPA-2940] Implements CONCAT operator

44d0d7f

[WIP][OPENJPA-2940] Implements NULLS FIRST / LAST

f90549c

[WIP][OPENJPA-2940] Implements set operations

ba0a2d6

[WIP][OPENJPA-2940] Make identification variables and SELECT clause o…

8d373a3

…ptional in JPQL" implementation

[WIP][OPENJPA-2940] More work on the JPQL / Criteria side

c138b83

[WIP][OPENJPA-2940] Embed Records

bc8926b

[WIP][OPENJPA-2940] ORDER BY alias regression (JPQL fix)

a5d7249

Fix Java 21 build

7ebc020

[WIP][OPENJPA-2940] Add additional tests for emebeddable records

5051dc7

[WIP][OPENJPA-2940] Add additional tests for emebeddable records and …

06cd8ec

…fixes

[WIP][OPENJPA-2940] Adds support for java.time.Instant and java.time.…

8753ab6

…Year a

[WIP][OPENJPA-2940] Adds support for Instant and LocalDateTime versions

b11656c

[WIP][OPENJPA-2940] Primary key classes are no longer required to be …

bf3d2b1

…public and serializable

[WIP][OPENJPA-2940] Static inner classes as entities/embeddables (alr…

d4ac0a5

…eady working)

[WIP][OPENJPA-2940] Handle jakarta.persistence.dataSource

f732f51

[WIP][OPENJPA-2940] Add secondPrecision to @column annotation support

1db7453

[WIP][OPENJPA-2940] Adds entities(), classes() and columns() to Named…

d99b28d

…NativeQuery

[WIP][OPENJPA-2940] Adds options member to elements which result in D…

00441ca

…DL generation

[WIP][OPENJPA-2940] Adds constants for managed types, named queries a…

b147889

…nd named result set mappings to generated StaticMetamodel

[WIP][OPENJPA-2940] Adds constants for managed types, named queries a…

52db2f0

…nd named result set mappings to generated StaticMetamodel

[WIP][OPENJPA-2940] Adds javax.annotation.processing.Generated to the…

816bed1

… list of defined annotations on StaticMetamodel

[WIP][OPENJPA-2940] Adds EnumeratedValue allowing custom mapping of f…

95f73af

…ields of Java enums

[WIP][OPENJPA-2940] Make the TCK runnable

3788f6f

[WIP][OPENJPA-2940] Implement JPA 2.1 ConstructorResult

0777e1f

[WIP][OPENJPA-2940] Make TCK runnable

88ff98e

[WIP][OPENJPA-2940] TCK: Fix Field/Property Access Type on TCK. If we…

d488eab

… have Property access only Getter/Setters according to Java Bean Style should work.

[WIP][OPENJPA-2940] TCK: JPA 3.2 spec Section 7.7: if close() is call…

2382fcb

…ed when a resource-local transaction is active, roll it back before closing. For managed transactions, defer closing until the transaction completes.

[WIP][OPENJPA-2940] TCK: Flush order for pre-existing FK keys in non …

d98a02c

…OpenJPA created schema

solomax and others added 8 commits August 18, 2026 13:01

[OPENJPA-2940] error param is fixed

6fadf68

[OPENJPA-2940] more work on Kubertenes tests

dec89b9

[OPENJPA-2940] VERSION() now orders by the version columns and reject…

e94a851

…s unversioned types with a clear message

[OPENJPA-2940] MetaDataRepository.addSystemListener/removeSystemListe…

e44b470

…ner are improved

Merge branch 'OPENJPA-2940' of github.com:apache/openjpa into OPENJPA…

02a1760

…-2940

[OPENJPA-2940] separate caches

ef99961

[OPENJPA-2940] more work on separate caches

ad060c6

[OPENJPA-2940] 3.2.x is added to the CI branches list

9c97ad4

[OPENJPA-2940] @EnumeratedValue: normalize codes, fail on unknown/dup…

0101aeb

…licate values, guard lazy init

[OPENJPA-2940] Oracle works as expected without sudo after switching …

4336f41

…to new container

solomax reviewed Aug 18, 2026

View reviewed changes

Comment thread ...istence-jdbc/src/test/java/org/apache/openjpa/persistence/criteria/TestTypesafeCriteria.java

solomax reviewed Aug 19, 2026

View reviewed changes

solomax added 3 commits August 19, 2026 11:44

[OPENJPA-2940] except for Oracle seems to be fixed

abec04b

[OPENJPA-2940] ExceptAll test is excluded for H2

ca985e2

[OPENJPA-2940] ExceptAll is not supported by MSSQL server

ba40303

cristof merged commit 5e6518e into master Aug 20, 2026
4 checks passed

solomax deleted the OPENJPA-2940 branch August 20, 2026 09:15

This was referenced Aug 20, 2026

[OPENJPA-2987] Restore externalized-parameter assertions #152

Merged

[OPENJPA-2971] Make PostgreSQL StoreCharsAsNumbers default overridable and release-note it #163

Merged

rzo1 mentioned this pull request Aug 25, 2026

[OPENJPA-2940] Migration considerations for OpenJPA 4.2.0 (Jakarta Persistence 3.2) #164

Open

Navigation Menu

Uh oh!

Implements Jakarta Persistence 3.2 - #144

Implements Jakarta Persistence 3.2#144
cristof merged 297 commits into
masterfrom
OPENJPA-2940

cristof commented May 16, 2026

Uh oh!

solomax commented Aug 18, 2026

Uh oh!

rzo1 commented Aug 18, 2026 •

edited

Loading

Uh oh!

solomax commented Aug 18, 2026

Uh oh!

rzo1 commented Aug 18, 2026 •

edited

Loading

Uh oh!

solomax commented Aug 18, 2026

Uh oh!

rzo1 commented Aug 18, 2026

Uh oh!

Uh oh!

solomax Aug 19, 2026

Uh oh!

solomax Aug 19, 2026

Uh oh!

cristof Aug 19, 2026

Uh oh!

rzo1 Aug 19, 2026

Uh oh!

solomax commented Aug 20, 2026

Uh oh!

cristof commented Aug 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Uh oh!

Conversation

cristof commented May 16, 2026

Uh oh!

solomax commented Aug 18, 2026

Uh oh!

rzo1 commented Aug 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

solomax commented Aug 18, 2026

Uh oh!

rzo1 commented Aug 18, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

solomax commented Aug 18, 2026

Uh oh!

rzo1 commented Aug 18, 2026

Uh oh!

Uh oh!

solomax Aug 19, 2026

Choose a reason for hiding this comment

Uh oh!

solomax Aug 19, 2026

Choose a reason for hiding this comment

Uh oh!

cristof Aug 19, 2026

Choose a reason for hiding this comment

Uh oh!

rzo1 Aug 19, 2026

Choose a reason for hiding this comment

Uh oh!

solomax commented Aug 20, 2026

Uh oh!

cristof commented Aug 20, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

rzo1 commented Aug 18, 2026 •

edited

Loading

rzo1 commented Aug 18, 2026 •

edited

Loading