Spring/spring-data-jpa
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Consider only top-level properties for tuple query selection.

Browse Source 
We now only consider top-level properties for tuple query selection to avoid join products caused by selecting nested relationships.

Closes #3908
This commit is contained in:
Mark Paluch
2025-06-04 11:21:55 +02:00
parent 5a22fbf467
commit 053a4620a0
1 changed files with 12 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

15
spring-data-jpa/src/main/java/org/springframework/data/jpa/repository/support/SimpleJpaRepository.java
View File

@@ -37,9 +37,11 @@ import java.util.ArrayList;
import java.util.Collection;
import java.util.Collections;
import java.util.HashMap;
import java.util.HashSet;
import java.util.List;
import java.util.Map;
import java.util.Optional;
import java.util.Set;
import java.util.function.BiConsumer;
import java.util.function.Function;
@@ -817,11 +819,18 @@ public class SimpleJpaRepository<T, ID> implements JpaRepositoryImplementation<T
}
List<Selection<?>> selections = new ArrayList<>();
Set<String> topLevelProperties = new HashSet<>();
for (String property : requiredSelection) {
PropertyPath path = PropertyPath.from(property, returnedType.getDomainType());
selections.add(QueryUtils.toExpressionRecursively(root, path, true).alias(property));
int separator = property.indexOf('.');
String topLevelProperty = separator == -1 ? property : property.substring(0, separator);
if (!topLevelProperties.add(topLevelProperty)) {
continue;
}
PropertyPath path = PropertyPath.from(topLevelProperty, returnedType.getDomainType());
selections.add(QueryUtils.toExpressionRecursively(root, path, true).alias(topLevelProperty));
}
Class<?> typeToRead = returnedType.getReturnedType();