Gavin King
Gavin King
@anija-anil would you do the favor of reviewing this one for me please?
I think we should merge this, because @sebersole is already working with this stuff on the implementation side and, well, it's essentially just mechanical work of syncing with the annotations....
Great, thanks for the review, Anija!
Him, thinking out load as usual, an option that looks maybe reasonable is to move all these constants to an inner `Properties` interface of `Persistence` (so `jakarta.persistence.Persistence.Properties`) and have `PersistenceConfiguration`...
> an option that looks maybe reasonable I have realized this idea in #862 and it turned out rather nicely. I think that's a good solution!
Hi, @MilaCodesIt, thanks for the offer of help. Actually I already implemented a proposal in the pull request https://github.com/jakartaee/persistence/pull/862, but I of course would appreciate a reviews of that PR...
This will be included in M1, so that we can get it in front of a wider audience.
> I think `CriteriaMutation` or `CriteriaModification` are good names Yes I think those names are fine.