HHH-19524 @OnetoOne relationship unnecessary joins in nativeQuery

Author: dreab8

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

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

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

@@ -1572,8 +1572,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,
@@ -1704,9 +1704,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/DomainResultCreationStateImpl.java

@@ -87,13 +87,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<String, DynamicFetchBuilderLegacy>> legacyFetchBuilders,
 			Consumer<SqlSelection> sqlSelectionConsumer,
 			LoadQueryInfluencers loadQueryInfluencers,
+			boolean isProcedureOrNativeQuery,
 			SessionFactoryImplementor sessionFactory) {
 		this.stateIdentifier = stateIdentifier;
 		this.jdbcResultsMetadata = jdbcResultsMetadata;
@@ -105,6 +107,8 @@ public DomainResultCreationStateImpl(
 		this.legacyFetchResolver = new LegacyFetchResolverImpl( legacyFetchBuilders );
 
 		this.sessionFactory = sessionFactory;
+
+		this.isProcedureOrNativeQuery = isProcedureOrNativeQuery;
 	}
 
 	public LegacyFetchResolver getLegacyFetchResolver() {
@@ -588,4 +592,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/ResultSetMappingImpl.java

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

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

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