From 74cc671087e833a6872221fbc97a3301520d1039 Mon Sep 17 00:00:00 2001 From: Henning Baldersheim Date: Wed, 20 Apr 2022 23:02:59 +0200 Subject: Sorting is a modifying operation => grab the children afterwards. --- .../src/main/java/com/yahoo/search/grouping/vespa/ResultBuilder.java | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'container-search/src/main/java') 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 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); -- cgit v1.2.3