summaryrefslogtreecommitdiffstats
path: root/container-search
diff options
context:
space:
mode:
authorHenning Baldersheim <balder@yahoo-inc.com>2022-04-20 23:02:59 +0200
committerHenning Baldersheim <balder@yahoo-inc.com>2022-04-20 23:02:59 +0200
commit74cc671087e833a6872221fbc97a3301520d1039 (patch)
tree7d32d54d6747a28bde785132873c26d973ced3cc /container-search
parent1514bb5390a138fe7b567957fcf21b31d75e10bb (diff)
Sorting is a modifying operation => grab the children afterwards.
Diffstat (limited to 'container-search')
-rw-r--r--container-search/src/main/java/com/yahoo/search/grouping/vespa/ResultBuilder.java2
1 files changed, 1 insertions, 1 deletions
diff --git a/container-search/src/main/java/com/yahoo/search/grouping/vespa/ResultBuilder.java b/container-search/src/main/java/com/yahoo/search/grouping/vespa/ResultBuilder.java
index ff029527f1c..7f006b098cd 100644
--- a/container-search/src/main/java/com/yahoo/search/grouping/vespa/ResultBuilder.java
+++ b/container-search/src/main/java/com/yahoo/search/grouping/vespa/ResultBuilder.java
@@ -326,9 +326,9 @@ class ResultBuilder {
void addGroup(com.yahoo.searchlib.aggregation.Group execGroup) {
GroupBuilder groupBuilder = getOrCreateGroup(execGroup);
if (execGroup.getNumChildren() > 0) {
+ execGroup.sortChildrenByRank();
List<com.yahoo.searchlib.aggregation.Group> children = execGroup.getChildren();
boolean ranked = children.get(0).isRankedByRelevance();
- execGroup.sortChildrenByRank();
for (com.yahoo.searchlib.aggregation.Group childGroup : children) {
GroupListBuilder childList = groupBuilder.getOrCreateChildList(childGroup.getTag(), ranked);
childList.addGroup(childGroup);