don't use a raw type in JdbcMapping

Author: gavinking

hibernate-core/src/main/java/org/hibernate/internal/SessionFactoryImpl.java

@@ -268,9 +268,9 @@ public SessionFactoryImpl(
 			final JdbcMapping jdbcMapping = tenantFilter.getParameterJdbcMapping( TenantIdBinder.PARAMETER_NAME );
 			assert jdbcMapping != null;
 			//noinspection unchecked
-			tenantIdentifierJavaType = jdbcMapping.getJavaTypeDescriptor();
+			tenantIdentifierJavaType = (JavaType<Object>) jdbcMapping.getJavaTypeDescriptor();
 		}
-		for ( Map.Entry<String, FilterDefinition> filterEntry : filters.entrySet() ) {
+		for ( var filterEntry : filters.entrySet() ) {
 			if ( filterEntry.getValue().isAutoEnabled() ) {
 				autoEnabledFilters.add( filterEntry.getValue() );
 			}

hibernate-core/src/main/java/org/hibernate/metamodel/mapping/JdbcMapping.java

@@ -54,7 +54,7 @@ public interface JdbcMapping extends MappingType, JdbcMappingContainer {
 	 * The descriptor for the Java type represented by this
 	 * expressible type
 	 */
-	JavaType getJavaTypeDescriptor();
+	JavaType<?> getJavaTypeDescriptor();
 
 	/**
 	 * The descriptor for the SQL type represented by this

hibernate-core/src/main/java/org/hibernate/query/results/internal/complete/CompleteResultBuilderBasicValuedConverted.java

@@ -46,14 +46,14 @@ public CompleteResultBuilderBasicValuedConverted(
 			BasicValuedMapping underlyingMapping) {
 		this.explicitColumnName = explicitColumnName;
 		this.underlyingMapping = underlyingMapping;
-		@SuppressWarnings("unchecked")
-		final JavaType<R> relationalType =
-				underlyingMapping.getJdbcMapping().getJavaTypeDescriptor();
-		this.valueConverter = new AttributeConverterBean<>(
+		final JavaType<?> relationalType =
+				underlyingMapping.getJdbcMapping()
+						.getJavaTypeDescriptor();
+		this.valueConverter = new JpaAttributeConverterImpl<>(
 				converterBean,
 				converterJtd,
 				domainJavaType,
-				relationalType
+				(JavaType<R>) relationalType
 		);
 	}
 

hibernate-core/src/main/java/org/hibernate/sql/ast/tree/expression/JdbcLiteral.java

@@ -78,7 +78,7 @@ public void bindParameterValue(
 	// MappingModelExpressible
 
 	@Override
-	public MappingModelExpressible<?> getExpressionType() {
+	public MappingModelExpressible<T> getExpressionType() {
 		return this;
 	}
 
@@ -184,6 +184,7 @@ public void applySqlSelections(DomainResultCreationState creationState) {
 
 	@Override
 	public JavaType<T> getExpressibleJavaType() {
-		return jdbcMapping.getJavaTypeDescriptor();
+		return (JavaType<T>)
+				jdbcMapping.getJavaTypeDescriptor();
 	}
 }

hibernate-core/src/main/java/org/hibernate/sql/exec/spi/AbstractJdbcOperationQuery.java

@@ -10,6 +10,7 @@
 
 import org.hibernate.query.spi.QueryOptions;
 import org.hibernate.sql.ast.tree.expression.JdbcParameter;
+import org.hibernate.type.descriptor.java.JavaType;
 
 import static java.util.Collections.emptyMap;
 
@@ -76,14 +77,18 @@ public boolean isCompatibleWith(JdbcParameterBindings jdbcParameterBindings, Que
 			for ( var entry : appliedParameters.entrySet() ) {
 				final JdbcParameterBinding binding = jdbcParameterBindings.getBinding( entry.getKey() );
 				final JdbcParameterBinding appliedBinding = entry.getValue();
-				//noinspection unchecked
-				if ( binding == null || !appliedBinding.getBindType()
-						.getJavaTypeDescriptor()
-						.areEqual( binding.getBindValue(), appliedBinding.getBindValue() ) ) {
+				if ( binding == null
+						|| !equal( appliedBinding, binding, appliedBinding.getBindType().getJavaTypeDescriptor() ) ) {
 					return false;
 				}
 			}
 		}
 		return true;
 	}
+
+	@SuppressWarnings("unchecked")
+	static <T> boolean equal(JdbcParameterBinding appliedBinding, JdbcParameterBinding binding, JavaType<T> type) {
+		return type.isInstance( appliedBinding.getBindValue() )
+			&& type.areEqual( (T) binding.getBindValue(), (T) appliedBinding.getBindValue() );
+	}
 }

hibernate-core/src/main/java/org/hibernate/sql/exec/spi/JdbcOperationQuerySelect.java

@@ -13,7 +13,6 @@
 import org.hibernate.query.spi.QueryOptions;
 import org.hibernate.sql.ast.tree.expression.JdbcParameter;
 import org.hibernate.sql.results.jdbc.spi.JdbcValuesMappingProducer;
-import org.hibernate.type.descriptor.java.JavaType;
 
 /**
  * Executable JDBC command
@@ -128,7 +127,8 @@ else if ( jdbcParameterBindings.getBinding( parameter ) == null ) {
 				if ( parameter != offsetParameter && parameter != limitParameter ) {
 					final JdbcParameterBinding binding = jdbcParameterBindings.getBinding( parameter );
 					// TODO: appliedBinding can be null here, resulting in NPE
-					if ( binding == null || !areEqualBindings( appliedBinding, binding ) ) {
+					if ( binding == null
+							|| !equal( appliedBinding, binding, appliedBinding.getBindType().getJavaTypeDescriptor() ) ) {
 						return false;
 					}
 				}
@@ -140,11 +140,6 @@ && isCompatible( offsetParameter, limit == null ? null : limit.getFirstRow(), 0
 			&& isCompatible( limitParameter, limit == null ? null : limit.getMaxRows(), Integer.MAX_VALUE );
 	}
 
-	private static boolean areEqualBindings(JdbcParameterBinding appliedBinding, JdbcParameterBinding binding) {
-		final JavaType<Object> javaTypeDescriptor = appliedBinding.getBindType().getJavaTypeDescriptor();
-		return javaTypeDescriptor.areEqual( binding.getBindValue(), appliedBinding.getBindValue() );
-	}
-
 	private boolean isCompatible(JdbcParameter parameter, Integer requestedValue, int defaultValue) {
 		if ( parameter == null ) {
 			return requestedValue == null;

hibernate-core/src/main/java/org/hibernate/sql/results/graph/basic/BasicResult.java

@@ -54,7 +54,8 @@ public BasicResult(
 		this(
 				jdbcValuesArrayPosition,
 				resultVariable,
-				jdbcMapping.getJavaTypeDescriptor(),
+				(JavaType<T>)
+						jdbcMapping.getJavaTypeDescriptor(),
 				(BasicValueConverter<T,?>)
 						jdbcMapping.getValueConverter(),
 				navigablePath,