Emmanuel Bernard

Results 29 comments of Emmanuel Bernard

Hey there, I think we discussed it in the past but I can't recall the reason why we did not execute. Have you explore the implications on the spec and...

Ah I see, this would be for `Pattern` specifically. I read too quickly.

@geoand could it be the changes @gwenneg did on the default GraalVM version. Not sure what the build is using.

`gerReference` was designed to get an object "shell" with no access to the database. It lets implementors to not implement this very useful optimisation and hence why you get some...

Can you try one thing @manoelcampos Add https://github.com/manoelcampos/hibernate-concurrency/blob/master/2-database-concurrency-panache/src/main/java/com/manoelcampos/server/model/Cliente.java#L23 `existente.version = cliente.version;` to make sure we get the client side version copied so we detect the conflict. Also replace https://github.com/manoelcampos/hibernate-concurrency/blob/master/2-database-concurrency-panache/src/main/java/com/manoelcampos/server/model/Cliente.java#L27 `existente.persist()`...

@Sanne @gsmet I am a bit lost but it seems that ``` Foo fooWithChangesAndOldVersion = ...; Foo foo = em.find(Foo.class, 1L); foo.title = fooWithChangesAndOldVersion.title; foo.version = fooWithChangesAndOldVersion.version; em.flush(); ``` Does...

I don't think it's ok no, at least I'm not sure. I need to discuss with @FroMage and @Sanne on this one.

Let me take a step back. `PanacheEntity` objects are where operations live. JPA `merge()` by default, will return a _copy_ of a detached entity. And the use case we want...

Option B is to not have full state merge and force a manual copy. ``` Foo existing = Foo.findById(foo.id); existing.title = foo.title; existing.version = foo.version; existing.author = foo.author; existing.merge(); //...

To clarify, in option B we would have a different behavior of Hibernate (or of the panache layer): i.e. let the version be overridden and checked before. Option C To...