HHH-15634 Lazy basic property does not get updated on change #5461
Conversation
There was a problem hiding this comment.
Seems that you accidently also fixed https://hibernate.atlassian.net/browse/HHH-10747 since a failure expected test org.hibernate.orm.test.bytecode.enhancement.lazy.LazyLoadingByEnhancerSetterTest is now running through 😄
| // we add also all the non lazy properties because dynamic update is false | ||
| for ( int i = 0; i < propertyLaziness.length; i++ ) { | ||
| if ( propertyLaziness[i] == false ) { | ||
| propsToUpdate[i] = true; |
There was a problem hiding this comment.
We should probably have a quick discussion about this next week with @sebersole to hear his thoughts on this topic.
…dirty checking, lazy basic properties are not upated when set to null
There was a problem hiding this comment.
I ran the reproducer in Quarkus (quarkusio/quarkus#28881) against Hibernate ORM 5.6 compiled with a backport of this patch (#5465), and I confirm it fixes all issues.
Thanks a lot @dreab8!
|
@beikov gave me a huge help :) by the way we have to check that the call of the |
|
Then thanks to @beikov too :) |
https://hibernate.atlassian.net/browse/HHH-15634