HHH-19524 @OnetoOne relationship unnecessary joins in nativeQuery

Author: dreab8

hibernate-core/src/main/java/org/hibernate/generator/values/internal/GeneratedValuesMappingProducer.java

@@ -49,6 +49,7 @@ public JdbcValuesMapping resolve(
 				null,
 				sqlSelections::add,
 				loadQueryInfluencers,
+				false,
 				sessionFactory
 		);
 

hibernate-core/src/main/java/org/hibernate/metamodel/mapping/internal/ToOneAttributeMapping.java

@@ -1543,8 +1543,8 @@ public static class EntityB {
 		else {
 			side = this.sideNature;
 		}
-
-		if ( ( fetchTiming == FetchTiming.IMMEDIATE && selected ) || needsJoinFetch( side ) ) {
+		if ( fetchTiming == FetchTiming.IMMEDIATE && selected
+			|| !creationState.getSqlAstCreationState().isProcedureOrNativeQuery() && needsJoinFetch( side ) ) {
 			final TableGroup tableGroup = determineTableGroupForFetch(
 					fetchablePath,
 					fetchParent,
@@ -1675,9 +1675,12 @@ else if ( hasNotFoundAction()
 
 	private boolean needsJoinFetch(ForeignKeyDescriptor.Nature side) {
 		if ( side == ForeignKeyDescriptor.Nature.TARGET ) {
-			// The target model part doesn't correspond to the identifier of the target entity mapping
-			// so we must eagerly fetch with a join (subselect would still cause problems).
+			// With composite identifier if the target model part doesn't correspond to the identifier of the target entity mapping
+			// we must eagerly fetch with a join (subselect would still cause problems).
 			final EntityIdentifierMapping identifier = entityMappingType.getIdentifierMapping();
+			if ( identifier instanceof BasicEntityIdentifierMappingImpl ) {
+				return false;
+			}
 			final ValuedModelPart targetPart = foreignKeyDescriptor.getTargetPart();
 			if ( identifier != targetPart ) {
 				// If the identifier and the target part of the same class, we can preserve laziness as deferred loading will still work

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

@@ -82,13 +82,15 @@ public class DomainResultCreationStateImpl
 	private boolean processingKeyFetches = false;
 	private boolean resolvingCircularFetch;
 	private ForeignKeyDescriptor.Nature currentlyResolvingForeignKeySide;
+	private boolean isProcedureOrNativeQuery;
 
 	public DomainResultCreationStateImpl(
 			String stateIdentifier,
 			JdbcValuesMetadata jdbcResultsMetadata,
 			Map<String, Map<Fetchable, LegacyFetchBuilder>> legacyFetchBuilders,
 			Consumer<SqlSelection> sqlSelectionConsumer,
 			LoadQueryInfluencers loadQueryInfluencers,
+			boolean isProcedureOrNativeQuery,
 			SessionFactoryImplementor sessionFactory) {
 		this.stateIdentifier = stateIdentifier;
 		this.jdbcResultsMetadata = jdbcResultsMetadata;
@@ -100,6 +102,8 @@ public DomainResultCreationStateImpl(
 		this.legacyFetchResolver = new LegacyFetchResolver( legacyFetchBuilders );
 
 		this.sessionFactory = sessionFactory;
+
+		this.isProcedureOrNativeQuery = isProcedureOrNativeQuery;
 	}
 
 	public SessionFactoryImplementor getSessionFactory() {
@@ -350,11 +354,8 @@ public Fetch visitIdentifierFetch(EntityResultGraphNode fetchParent) {
 		final EntityIdentifierMapping identifierMapping = parentModelPart.getEntityMappingType().getIdentifierMapping();
 		final String identifierAttributeName = attributeName( identifierMapping );
 
-		//noinspection unchecked
-		final FetchBuilder explicitFetchBuilder = (FetchBuilder) fetchBuilderResolverStack
-				.getCurrent()
-				.apply( identifierMapping );
-		LegacyFetchBuilder fetchBuilderLegacy;
+		final FetchBuilder explicitFetchBuilder = fetchBuilderResolverStack.getCurrent().apply( identifierMapping );
+		final LegacyFetchBuilder fetchBuilderLegacy;
 		if ( explicitFetchBuilder == null ) {
 			fetchBuilderLegacy = legacyFetchResolver.resolve(
 					fromClauseAccess.findTableGroup( fetchParent.getNavigablePath() )
@@ -414,9 +415,7 @@ private Consumer<Fetchable> createFetchableConsumer(FetchParent fetchParent, Imm
 			if ( !fetchable.isSelectable() ) {
 				return;
 			}
-			FetchBuilder explicitFetchBuilder = (FetchBuilder) fetchBuilderResolverStack
-					.getCurrent()
-					.apply( fetchable );
+			FetchBuilder explicitFetchBuilder = fetchBuilderResolverStack.getCurrent().apply( fetchable );
 			LegacyFetchBuilder fetchBuilderLegacy;
 			if ( explicitFetchBuilder == null ) {
 				fetchBuilderLegacy = legacyFetchResolver.resolve(
@@ -490,4 +489,8 @@ public void setCurrentlyResolvingForeignKeyPart(ForeignKeyDescriptor.Nature curr
 		this.currentlyResolvingForeignKeySide = currentlyResolvingForeignKeySide;
 	}
 
+	@Override
+	public boolean isProcedureOrNativeQuery() {
+		return isProcedureOrNativeQuery;
+	}
 }

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

@@ -199,6 +199,7 @@ public JdbcValuesMapping resolve(
 				legacyFetchBuilders,
 				sqlSelections::add,
 				loadQueryInfluencers,
+				true,
 				sessionFactory
 		);
 

hibernate-core/src/main/java/org/hibernate/sql/ast/spi/SqlAstCreationState.java

@@ -57,4 +57,8 @@ default boolean supportsEntityNameUsage() {
 	@Internal
 	default void applyOrdering(TableGroup tableGroup, OrderByFragment orderByFragment) {
 	}
+
+	default boolean isProcedureOrNativeQuery(){
+		return false;
+	}
 }