[#1717] Upgrade Hibernate ORM to 6.2.7.Final

Fix compilation errors and test failure.

`BasicTypesAndCallbacksForAllDBsTest#testCallbacksAndVersioning`
fails because Hibernate ORM now throws an exception when the
associated entity has null version.

Becaue in our test the association is lazy, it means we need to
fetch it before removing the entity owner.

See the related change in Hibernate ORM:
hibernate/hibernate-orm@1653023

I've also added some assertions to check how the version value
changes in the different steps.

Author: DavideD

build.gradle

@@ -53,7 +53,7 @@ version = projectVersion
 // ./gradlew clean build -PhibernateOrmVersion=5.6.15-SNAPSHOT
 ext {
     if ( !project.hasProperty('hibernateOrmVersion') ) {
-        hibernateOrmVersion = '6.2.6.Final'
+        hibernateOrmVersion = '6.2.7.Final'
     }
     if ( !project.hasProperty( 'hibernateOrmGradlePluginVersion' ) ) {
         // Same as ORM as default

hibernate-reactive-core/src/main/java/org/hibernate/reactive/persister/collection/impl/ReactiveBasicCollectionPersister.java

@@ -110,7 +110,12 @@ private ReactiveDeleteRowsCoordinator buildDeleteRowCoordinator() {
 			return new ReactiveDeleteRowsCoordinatorNoOp( this );
 		}
 
-		return new ReactiveDeleteRowsCoordinatorStandard( this, getRowMutationOperations(), hasPhysicalIndexColumn() );
+		return new ReactiveDeleteRowsCoordinatorStandard(
+				this,
+				getRowMutationOperations(),
+				hasPhysicalIndexColumn(),
+				getFactory().getServiceRegistry()
+		);
 	}
 
 	private ReactiveRemoveCoordinator buildDeleteAllCoordinator() {
@@ -121,7 +126,11 @@ private ReactiveRemoveCoordinator buildDeleteAllCoordinator() {
 			return new ReactiveRemoveCoordinatorNoOp( this );
 		}
 
-		return new ReactiveRemoveCoordinatorStandard( this, this::buildDeleteAllOperation );
+		return new ReactiveRemoveCoordinatorStandard(
+				this,
+				this::buildDeleteAllOperation,
+				getFactory().getServiceRegistry()
+		);
 	}
 
 	@Override

hibernate-reactive-core/src/main/java/org/hibernate/reactive/persister/collection/impl/ReactiveOneToManyPersister.java

@@ -124,7 +124,12 @@ private ReactiveDeleteRowsCoordinator buildDeleteCoordinator() {
 			return new ReactiveDeleteRowsCoordinatorNoOp( this );
 		}
 		// never delete by index for one-to-many
-		return new ReactiveDeleteRowsCoordinatorStandard( this, getRowMutationOperations(), false );
+		return new ReactiveDeleteRowsCoordinatorStandard(
+				this,
+				getRowMutationOperations(),
+				false,
+				getFactory().getServiceRegistry()
+		);
 	}
 
 	private ReactiveRemoveCoordinator buildDeleteAllCoordinator() {
@@ -134,7 +139,11 @@ private ReactiveRemoveCoordinator buildDeleteAllCoordinator() {
 			}
 			return new ReactiveRemoveCoordinatorNoOp( this );
 		}
-		return new ReactiveRemoveCoordinatorStandard( this, this::buildDeleteAllOperation );
+		return new ReactiveRemoveCoordinatorStandard(
+				this,
+				this::buildDeleteAllOperation,
+				getFactory().getServiceRegistry()
+		);
 	}
 
 	@Override

hibernate-reactive-core/src/main/java/org/hibernate/reactive/persister/collection/mutation/ReactiveDeleteRowsCoordinatorStandard.java

@@ -19,6 +19,7 @@
 import org.hibernate.persister.collection.mutation.DeleteRowsCoordinatorStandard;
 import org.hibernate.persister.collection.mutation.RowMutationOperations;
 import org.hibernate.reactive.engine.jdbc.env.internal.ReactiveMutationExecutor;
+import org.hibernate.service.ServiceRegistry;
 import org.hibernate.sql.model.MutationOperationGroup;
 import org.hibernate.sql.model.MutationType;
 import org.hibernate.sql.model.internal.MutationOperationGroupSingle;
@@ -35,8 +36,12 @@ public class ReactiveDeleteRowsCoordinatorStandard extends DeleteRowsCoordinator
 	private MutationOperationGroupSingle operationGroup;
 	private final BasicBatchKey batchKey;
 
-	public ReactiveDeleteRowsCoordinatorStandard(CollectionMutationTarget mutationTarget, RowMutationOperations rowMutationOperations, boolean deleteByIndex) {
-		super( mutationTarget, rowMutationOperations, deleteByIndex );
+	public ReactiveDeleteRowsCoordinatorStandard(
+			CollectionMutationTarget mutationTarget,
+			RowMutationOperations rowMutationOperations,
+			boolean deleteByIndex,
+			ServiceRegistry serviceRegistry) {
+		super( mutationTarget, rowMutationOperations, deleteByIndex, serviceRegistry );
 		this.deleteByIndex = deleteByIndex;
 		this.rowMutationOperations = rowMutationOperations;
 		this.batchKey = new BasicBatchKey( mutationTarget.getRolePath() + "#DELETE" );

hibernate-reactive-core/src/main/java/org/hibernate/reactive/persister/collection/mutation/ReactiveRemoveCoordinatorStandard.java

@@ -21,6 +21,7 @@
 import org.hibernate.reactive.logging.impl.Log;
 import org.hibernate.reactive.logging.impl.LoggerFactory;
 import org.hibernate.reactive.util.impl.CompletionStages;
+import org.hibernate.service.ServiceRegistry;
 import org.hibernate.sql.model.MutationOperationGroup;
 import org.hibernate.sql.model.MutationType;
 import org.hibernate.sql.model.ast.MutatingTableReference;
@@ -40,8 +41,11 @@ public class ReactiveRemoveCoordinatorStandard extends RemoveCoordinatorStandard
 	private final OperationProducer operationProducer;
 	private MutationOperationGroupSingle operationGroup;
 
-	public ReactiveRemoveCoordinatorStandard(CollectionMutationTarget mutationTarget, OperationProducer operationProducer) {
-		super( mutationTarget, operationProducer );
+	public ReactiveRemoveCoordinatorStandard(
+			CollectionMutationTarget mutationTarget,
+			OperationProducer operationProducer,
+			ServiceRegistry serviceRegistry) {
+		super( mutationTarget, operationProducer, serviceRegistry );
 		this.batchKey = new BasicBatchKey( mutationTarget.getRolePath() + "#REMOVE" );
 		this.operationProducer = operationProducer;
 	}