diff options
author | Henning Baldersheim <balder@yahoo-inc.com> | 2022-04-20 23:02:59 +0200 |
---|---|---|
committer | Henning Baldersheim <balder@yahoo-inc.com> | 2022-04-20 23:02:59 +0200 |
commit | 74cc671087e833a6872221fbc97a3301520d1039 (patch) | |
tree | 7d32d54d6747a28bde785132873c26d973ced3cc /container-search | |
parent | 1514bb5390a138fe7b567957fcf21b31d75e10bb (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.java | 2 |
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); |