Barry LaFond
Barry LaFond
[maresja1](https://github.com/maresja1) can you add a reproducer and/or test scenario with entities?
Been digging around to try and find a proper test case or scenario. The error is being thrown via the reactive [AbstractReactiveSaveEventListener.reactivePerformSave() ](https://github.com/hibernate/hibernate-reactive/blob/8debb26be219c80c04e1b849c807d1d7eb87a7da/hibernate-reactive-core/src/main/java/org/hibernate/reactive/event/impl/AbstractReactiveSaveEventListener.java#L172)which basically contains the same code as ORM's...
After further testing in ORM, there are a number of tests that create a DELETED entity status where the entity is "old", however, I can't find a test/scenario where the...
I ran your in test with an ORM test in hibernate-core ``` Caused by: org.hibernate.PersistentObjectException: detached entity passed to persist: org.hibernate.test.any.Person at org.hibernate.event.internal.DefaultPersistEventListener.onPersist(DefaultPersistEventListener.java:120) at org.hibernate.event.internal.DefaultPersistEventListener.onPersist(DefaultPersistEventListener.java:55) at org.hibernate.event.service.internal.EventListenerGroupImpl.fireEventOnEachListener(EventListenerGroupImpl.java:107) at org.hibernate.internal.SessionImpl.firePersist(SessionImpl.java:756) ......
@DavideD The commits above fixes the issue. I still need to refactor the `reactivePerformSave()` to simplify it.
@vietj Not sure why the conversion lower case other than to maybe simplify the `switch(key)` logic. Seems to me that each XXConnectOptions implementation should adhere to the case of each...
From a hibernate-reactive standpoint, we're intending to allow a user to pass along any syntax/valid property on the URL. Problem with properties across DB types is there are few "common"...
Looks like the `Driver` classes consume the `SqlConnectOptions` which are used to create the `PoolImpl`. So `JsonObject` maintains a "configuration". Is that object sent outside the Vertx clients? Just wondering...
Agreed. The commits (above) to my fork capture the extra properties during parsing and adds them to the configuration properties map. However, when applied via an Hibernate Reactive connection URL,...
@vietj I made some changes locally to just log a warning. Here's an example of what a warning log entry might look like: ``` INFO DefaultSqlClientPool [vert.x-worker-thread-0] HR000011: SQL Client...