hibernate / hibernate-reactive

A reactive API for Hibernate ORM, supporting non-blocking database drivers and a reactive style of interaction with the database.
https://hibernate.org/reactive
Apache License 2.0
424 stars 88 forks source link

Reactive find with lock in Quarkus with reactive hibernate - Issue reproduced #1905

Open alexjaravete opened 1 month ago

alexjaravete commented 1 month ago

Hi, this is related to this issue https://github.com/hibernate/hibernate-reactive/issues/1896.

Hi, when using Quarkus with reactive hibernate and trying to run this findById(id, LockModeType.PESSIMISTIC_WRITE), I get following Error: JPA error: java.lang.UnsupportedOperationException The error seems to be coming from here: hibernate-reactive/hibernate-reactive-core/src/main/java/org/hibernate/reactive/event/impl/DefaultReactiveLockEventListener.java Lines 238 to 241 in 5cb015d @Override public void onLock(LockEvent event) throws HibernateException { throw new UnsupportedOperationException(); } Is this intended, or is the implementation not finished? I can see that the last time this was edited was 4 years ago.

I was able to reproduce the issue when using a OneToMany and ManyToOne relationship. By doing this, hibernate is forced to do a JOIN and the SELECT FOR UDPATE cannot be done in the same request. When hibernate is trying to do the followup Lock, it throws an error.

The error should already show up after creating a Fruit and fetching it. No Owner object is needed. It's only there to trigger the issue.

This is the repository with the reproduced issue: https://github.com/alexjaravete/quarkus-quickstarts/tree/findById/hibernate-reactive-panache-quickstart

DavideD commented 1 month ago

Thanks for the project. I can see the error.

It seems quite complicated to solve, but I will give it a try