diff options
Diffstat (limited to 'container-search/src/main/java/com/yahoo/prelude/fastsearch/DocsumDefinitionSet.java')
-rw-r--r-- | container-search/src/main/java/com/yahoo/prelude/fastsearch/DocsumDefinitionSet.java | 24 |
1 files changed, 5 insertions, 19 deletions
diff --git a/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocsumDefinitionSet.java b/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocsumDefinitionSet.java index 6f965a61b16..9962c804b7e 100644 --- a/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocsumDefinitionSet.java +++ b/container-search/src/main/java/com/yahoo/prelude/fastsearch/DocsumDefinitionSet.java @@ -26,7 +26,6 @@ public final class DocsumDefinitionSet { public static final int SLIME_MAGIC_ID = 0x55555555; private final static Logger log = Logger.getLogger(DocsumDefinitionSet.class.getName()); - private final HashMap<Long, DocsumDefinition> definitions = new HashMap<>(); private final HashMap<String, DocsumDefinition> definitionsByName = new HashMap<>(); private final LegacyEmulationConfig emulationConfig; @@ -40,14 +39,6 @@ public final class DocsumDefinitionSet { configure(config); } - /** Returns a docsum definition by id - * @param id document summary class id - * @return a DocsumDefinition for the id, if found. - */ - public final DocsumDefinition getDocsumDefinition(long id) { - return definitions.get(new Long(id)); - } - /** * Returns a docsum definition by name, or null if not found * @@ -95,32 +86,27 @@ public final class DocsumDefinitionSet { public String toString() { StringBuilder sb = new StringBuilder(); - Set<Map.Entry<Long, DocsumDefinition>> entrySet = definitions.entrySet(); boolean first = true; - for (Iterator<Map.Entry<Long, DocsumDefinition>> itr = entrySet.iterator(); itr.hasNext(); ) { - if (!first) { + for (Map.Entry<String, DocsumDefinition> e : definitionsByName.entrySet() ) { + if (sb.length() != 0) { sb.append(","); - } else { - first = false; } - Map.Entry<Long, DocsumDefinition> entry = itr.next(); - sb.append("[").append(entry.getKey()).append(",").append(entry.getValue().getName()).append("]"); + sb.append("[").append(e.getKey()).append(",").append(e.getValue().getName()).append("]"); } return sb.toString(); } public int size() { - return definitions.size(); + return definitionsByName.size(); } private void configure(DocumentdbInfoConfig.Documentdb config) { for (int i = 0; i < config.summaryclass().size(); ++i) { DocumentdbInfoConfig.Documentdb.Summaryclass sc = config.summaryclass(i); DocsumDefinition docSumDef = new DocsumDefinition(sc, emulationConfig); - definitions.put((long) sc.id(), docSumDef); definitionsByName.put(sc.name(), docSumDef); } - if (definitions.size() == 0) { + if (definitionsByName.size() == 0) { log.warning("No summary classes found in DocumentdbInfoConfig.Documentdb"); } } |