[#1799] Upgrade Hibernate ORM to 6.3.2.Final

Author: DavideD

README.md

@@ -37,7 +37,7 @@ Hibernate Reactive has been tested with:
 - CockroachDB 22.1
 - MS SQL Server 2019
 - Oracle 21.3
-- [Hibernate ORM][] 6.3.1.Final
+- [Hibernate ORM][] 6.3.2.Final
 - [Vert.x Reactive PostgreSQL Client](https://vertx.io/docs/vertx-pg-client/java/) 4.5.0
 - [Vert.x Reactive MySQL Client](https://vertx.io/docs/vertx-mysql-client/java/) 4.5.0
 - [Vert.x Reactive Db2 Client](https://vertx.io/docs/vertx-db2-client/java/) 4.5.0

build.gradle

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

hibernate-reactive-core/src/main/java/org/hibernate/reactive/engine/impl/ReactiveEntityUpdateAction.java

@@ -18,7 +18,6 @@
 import org.hibernate.reactive.engine.ReactiveExecutable;
 import org.hibernate.reactive.persister.entity.impl.ReactiveEntityPersister;
 import org.hibernate.stat.spi.StatisticsImplementor;
-import org.hibernate.tuple.entity.EntityMetamodel;
 import org.hibernate.type.TypeHelper;
 
 import static org.hibernate.reactive.util.impl.CompletionStages.completedFuture;
@@ -135,21 +134,6 @@ private CompletionStage<EntityEntry> handleGeneratedProperties(EntityEntry entry
 		}
 	}
 
-	// TODO: copy/paste from superclass (make it protected)
-	private void handleDeleted(EntityEntry entry) {
-		if ( entry.getStatus() == Status.DELETED ) {
-			final EntityMetamodel entityMetamodel = getPersister().getEntityMetamodel();
-			final boolean isImpliedOptimisticLocking = !entityMetamodel.isVersioned()
-					&& entityMetamodel.getOptimisticLockStyle().isAllOrDirty();
-			if ( isImpliedOptimisticLocking && entry.getLoadedState() != null ) {
-				// The entity will be deleted and because we are going to create a delete statement
-				// that uses all the state values in the where clause, the entry state needs to be
-				// updated otherwise the statement execution will not delete any row (see HHH-15218).
-				entry.postUpdate( getInstance(), getState(), getNextVersion() );
-			}
-		}
-	}
-
 	private CompletionStage<Void> processGeneratedProperties(
 			Object id,
 			ReactiveEntityPersister persister,

hibernate-reactive-core/src/main/java/org/hibernate/reactive/loader/ast/internal/ReactiveSingleIdArrayLoadPlan.java

@@ -9,6 +9,7 @@
 
 import org.hibernate.LockOptions;
 import org.hibernate.engine.spi.SessionFactoryImplementor;
+import org.hibernate.metamodel.mapping.EntityMappingType;
 import org.hibernate.metamodel.mapping.ModelPart;
 import org.hibernate.sql.ast.tree.select.SelectStatement;
 import org.hibernate.sql.exec.spi.JdbcParametersList;
@@ -21,12 +22,13 @@
 public class ReactiveSingleIdArrayLoadPlan extends ReactiveSingleIdLoadPlan<Object[]> {
 
 	public ReactiveSingleIdArrayLoadPlan(
+			EntityMappingType entityMappingType,
 			ModelPart restrictivePart,
 			SelectStatement sqlAst,
 			JdbcParametersList jdbcParameters,
 			LockOptions lockOptions,
 			SessionFactoryImplementor sessionFactory) {
-		super( null, restrictivePart, sqlAst, jdbcParameters, lockOptions, sessionFactory );
+		super( entityMappingType, restrictivePart, sqlAst, jdbcParameters, lockOptions, sessionFactory );
 	}
 
 	@Override

hibernate-reactive-core/src/main/java/org/hibernate/reactive/loader/ast/internal/ReactiveSingleIdLoadPlan.java

@@ -63,7 +63,15 @@ public CompletionStage<T> load(Object restrictedValue, Object entityInstance, Bo
 		assert offset == getJdbcParameters().size();
 		final QueryOptions queryOptions = new SimpleQueryOptions( getLockOptions(), readOnly );
 		final Callback callback = new CallbackImpl();
-		ExecutionContext executionContext = executionContext( restrictedValue, entityInstance, session, queryOptions, callback );
+		EntityMappingType loadable = (EntityMappingType) getLoadable();
+		ExecutionContext executionContext = executionContext(
+				restrictedValue,
+				entityInstance,
+				session,
+				loadable.getRootEntityDescriptor(),
+				queryOptions,
+				callback
+		);
 		// FIXME: Should we get this from jdbcServices.getSelectExecutor()?
 		return StandardReactiveSelectExecutor.INSTANCE
 				.list( getJdbcSelect(), jdbcParameterBindings, executionContext, getRowTransformer(), resultConsumer( singleResultExpected ) )
@@ -88,6 +96,7 @@ private static ExecutionContext executionContext(
 			Object restrictedValue,
 			Object entityInstance,
 			SharedSessionContractImplementor session,
+			EntityMappingType rootEntityDescriptor,
 			QueryOptions queryOptions,
 			Callback callback) {
 		return new ExecutionContext() {
@@ -151,6 +160,11 @@ public QueryParameterBindings getQueryParameterBindings() {
 				return QueryParameterBindings.NO_PARAM_BINDINGS;
 			}
 
+			@Override
+			public EntityMappingType getRootEntityDescriptor() {
+				return rootEntityDescriptor;
+			}
+
 			@Override
 			public Callback getCallback() {
 				return callback;

hibernate-reactive-core/src/main/java/org/hibernate/reactive/persister/entity/impl/ReactiveAbstractEntityPersister.java

@@ -629,7 +629,7 @@ default ReactiveSingleIdArrayLoadPlan createLazyLoadPlan(List<LazyAttributeDescr
 					factory
 			);
 			final JdbcParametersList jdbcParameters = jdbcParametersListBuilder.build();
-			return new ReactiveSingleIdArrayLoadPlan( getIdentifierMapping(), select, jdbcParameters, LockOptions.NONE, factory );
+			return new ReactiveSingleIdArrayLoadPlan( this, getIdentifierMapping(), select, jdbcParameters, LockOptions.NONE, factory );
 		}
 	}