Revert "Merge pull request #5663 from vespa-engine/revert-5628-bratseth/dispatch-summaries-take-4"

This reverts commit dc350f9b5b3ea6ec73cb03f9e1a9feee9aedb754, reversing
changes made to 416381813ecc71079cebdfc2fbea53db50714014.

1 files changed, 19 insertions, 9 deletions

diff --git a/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocumentDatabase.java b/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocumentDatabase.java
index 3366f92384a..0ae0983a1ae 100644
--- a/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocumentDatabase.java
+++ b/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocumentDatabase.java
@@ -4,10 +4,13 @@ package com.yahoo.prelude.fastsearch;
 import com.google.common.collect.ImmutableMap;
 import com.yahoo.container.search.LegacyEmulationConfig;
 
+import java.util.ArrayList;
+import java.util.Collection;
 import java.util.Collections;
 import java.util.HashMap;
 import java.util.List;
 import java.util.Map;
+import java.util.stream.Collectors;
 
 /**
  * Representation of a back-end document database.
@@ -24,12 +27,16 @@ public class DocumentDatabase {
     private final String name;
     private final DocsumDefinitionSet docsumDefSet;
 
-    private final Map<String, RankProfile> rankProfiles;
+    private final ImmutableMap<String, RankProfile> rankProfiles;
 
     public DocumentDatabase(DocumentdbInfoConfig.Documentdb documentDb, LegacyEmulationConfig emulConfig) {
-        this.name = documentDb.name();
-        this.docsumDefSet = new DocsumDefinitionSet(documentDb, emulConfig);
-        this.rankProfiles = ImmutableMap.copyOf(toRankProfiles(documentDb.rankprofile()));
+        this(documentDb.name(), new DocsumDefinitionSet(documentDb, emulConfig), toRankProfiles(documentDb.rankprofile()));
+    }
+
+    public DocumentDatabase(String name, DocsumDefinitionSet docsumDefinitionSet, Collection<RankProfile> rankProfiles) {
+        this.name = name;
+        this.docsumDefSet = docsumDefinitionSet;
+        this.rankProfiles = ImmutableMap.copyOf(rankProfiles.stream().collect(Collectors.toMap(RankProfile::getName, p -> p)));
     }
 
     public String getName() {
@@ -43,11 +50,14 @@ public class DocumentDatabase {
     /** Returns an unmodifiable map of all the rank profiles in this indexed by rank profile name */
     public Map<String, RankProfile> rankProfiles() { return rankProfiles; }
 
-    private Map<String, RankProfile> toRankProfiles(List<DocumentdbInfoConfig.Documentdb.Rankprofile> rankProfileConfigList) {
-        Map<String, RankProfile> rankProfiles = new HashMap<>();
-        for (DocumentdbInfoConfig.Documentdb.Rankprofile c : rankProfileConfigList) {
-            rankProfiles.put(c.name(), new RankProfile(c.name(), c.hasSummaryFeatures(), c.hasRankFeatures()));
-        }
+    private static ImmutableMap<String, RankProfile> toMap(Collection<RankProfile> rankProfiles) {
+        return ImmutableMap.copyOf(rankProfiles.stream().collect(Collectors.toMap(RankProfile::getName, p -> p)));
+    }
+
+    private static Collection<RankProfile> toRankProfiles(Collection<DocumentdbInfoConfig.Documentdb.Rankprofile> rankProfileConfigList) {
+        List<RankProfile> rankProfiles = new ArrayList<>();
+        for (DocumentdbInfoConfig.Documentdb.Rankprofile c : rankProfileConfigList)
+            rankProfiles.add(new RankProfile(c.name(), c.hasSummaryFeatures(), c.hasRankFeatures()));
         return rankProfiles;
     }