Andy Jefferson

+1 for auto-quoting database identifiers when necessary to avoid use of SQL keywords (*not* adding some Hibernate property), but then other persistence providers (e.g DataNucleus) have done that for years...

-1 In the same way EclipseLink has its own way of auditing, and other providers have their own ways. Any standardisation would be of an API (and associated specification metadata),...

These annotations should be applicable for fields of type java.sql.Timestamp, or java.time.Instant.

FWIW java.time.Instant would be a much more useful addition here.

There are two separate things here. 1. Support specification of `@Convert` on an "id" attribute. 2. Support use of `@Convert` when an attribute is generated, and define what is the...

getNumberOfSubExpressions, getSubExpression, isParameter, invoke were added as delegated back in this commit https://github.com/datanucleus/datanucleus-rdbms/commit/dddee142870e086dae5647845d65a93bb5c2b209 Since no testcase is provided for this issue then nothing to say about whether it "fixes" it...

Impossible to comment on something that can't be easily seen. Kindly follow https://www.datanucleus.org/documentation/problem_reporting.html#jpa

Some test suite failing is not a bug. A bug is some functionality fails. Suggest that you debug the situation and isolate it to some actual operation. No resource here...

Thanks for your comments. You are welcome to contribute a proposed improvement to the handling, via a pull request. Yes, the current maxSize handling is simple; it provides a reference...