-
Notifications
You must be signed in to change notification settings - Fork 368
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add module for hibernate 6.4 #691
Conversation
@adrianAzoitei Thanks. I'll review it when I have some time. |
@adrianAzoitei There are many issues:
The problem seems to be in Hibernate since any You could add a Hibernate JIRA issue for this as there may be devs interested in supplying their own custom |
Thanks for the suggestion, @vladmihalcea. Spent some time trying to figure out if we can work around this issue in this library, but didn’t manage to come up with something. Do you think that’s possible? If not, what’s your take? Postpone hypersistence-utils-64? Release it without array support? |
You should open a Hibernate issue about this since it's because of the changes in Hibernate 6.4 that UserType can no longer be used for plural types. After you create the issue, you can discuss it on the Hibernate Forum. Afterward, if Hibernate 6.4 will allow a UserType to be used as a PluralType, then it will be possible to port the library to 6.4. |
For the record:
(I also stumbled upon this issue and had to search for those to see if there were any updates.) |
@DManstrator The link to the Hibernate forum is on the issue. This is just a Pull Request. It's better to have all the discussions related to this topic on the issue, and not on the Pull Request. |
Not much seems to be needed for this module, after the work done in #684.
Fixes #685.