Skip to content
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

From Hibernate 6.2, both MutableType and ImmutableType need to implement BasicDomainType #609

Open
vladmihalcea opened this issue Apr 14, 2023 · 3 comments

Comments

@vladmihalcea
Copy link
Owner

vladmihalcea commented Apr 14, 2023

Without implementing BasicDomainType, we won't be able to pass the MutableType and ImmutableType instance to addScalar.

However, this is a gigantic task as all existing Types that now extend the MutableType and ImmutableType would have to use JavaType and JdbcType descriptors and also extend BasicType. Unless a company is willing to sponsor this issue, I doubt that I will have the time to implement it in the next 1-2 years.

@vladmihalcea
Copy link
Owner Author

vladmihalcea commented Apr 16, 2023

This will require using the MutableDynamicParameterizedType, like the Json Types.

@vladmihalcea vladmihalcea added this to the 3.3.2 milestone Apr 16, 2023
@vladmihalcea vladmihalcea reopened this Apr 16, 2023
@vladmihalcea vladmihalcea removed this from the 3.3.2 milestone Apr 16, 2023
@radtriste
Copy link

Hi @vladmihalcea
Facing same issue with JsonBlobType and JsonNodeBinaryType in addScalar method.
I am quite new with your library and we are upgrading our project to Hibernate 6.2
Not sure if there is currently a workaround ?
Or do you have any ETA for that issue ?

@vladmihalcea
Copy link
Owner Author

vladmihalcea commented May 15, 2023

There's no ETA for it. At the moment, I'm busy with other projects, and I won't have the time to investigate it.

However, the beauty of OSS is that you don't depend on my agenda. If this feature is a blocker, I'm sure someone will invest time into providing a solution.

Looking forward to the Pull Request. Be the change you want to see in the world!

@vladmihalcea vladmihalcea changed the title From Hibernate 6.2, ImmutableType needs to implement BasicDomainType From Hibernate 6.2, MutableType and ImmutableType need to implement BasicDomainType Jun 17, 2023
@vladmihalcea vladmihalcea changed the title From Hibernate 6.2, MutableType and ImmutableType need to implement BasicDomainType From Hibernate 6.2, ImmutableType need to implement BasicDomainType Jun 17, 2023
@vladmihalcea vladmihalcea changed the title From Hibernate 6.2, ImmutableType need to implement BasicDomainType From Hibernate 6.2, both MutableType and ImmutableType need to implement BasicDomainType Jun 17, 2023
nrayburn-tech pushed a commit to nrayburn-tech/hypersistence-utils that referenced this issue Oct 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants