Ensure that sameElement never accepts a WordAlternativesItem.

3 files changed, 1 insertions, 24 deletions

diff --git a/container-search/src/main/java/com/yahoo/prelude/query/SameElementItem.java b/container-search/src/main/java/com/yahoo/prelude/query/SameElementItem.java
index ac4e8b98b03..58bbcd7315c 100644
--- a/container-search/src/main/java/com/yahoo/prelude/query/SameElementItem.java
+++ b/container-search/src/main/java/com/yahoo/prelude/query/SameElementItem.java
@@ -50,6 +50,7 @@ public class SameElementItem extends NonReducibleCompositeItem {
         super.adding(item);
         //TODO See if we can require only SimpleIndexedItem instead of TermItem
         Validator.ensureInstanceOf("Child item", item, TermItem.class);
+        Validator.ensureNotInstanceOf("Child item", item, WordAlternativesItem.class);
         TermItem asTerm = (TermItem) item;
         Validator.ensureNonEmpty("Struct fieldname", asTerm.getIndexName());
         Validator.ensureNonEmpty("Query term", asTerm.getIndexedString());
diff --git a/container-search/src/main/java/com/yahoo/prelude/query/WordAlternativesItem.java b/container-search/src/main/java/com/yahoo/prelude/query/WordAlternativesItem.java
index 2c017410109..d2df2aa6c89 100644
--- a/container-search/src/main/java/com/yahoo/prelude/query/WordAlternativesItem.java
+++ b/container-search/src/main/java/com/yahoo/prelude/query/WordAlternativesItem.java
@@ -19,7 +19,6 @@ import com.yahoo.compress.IntegerCompressor;
 public class WordAlternativesItem extends TermItem {
 
     private List<Alternative> alternatives;
-    private int maxIndex;
 
     public static final class Alternative {
 
@@ -48,7 +47,6 @@ public class WordAlternativesItem extends TermItem {
 
     public void setAlternatives(Collection<Alternative> terms) {
         this.alternatives = uniqueAlternatives(terms);
-        setMaxIndex();
     }
 
     private static ImmutableList<Alternative> uniqueAlternatives(Collection<Alternative> terms) {
@@ -67,27 +65,6 @@ public class WordAlternativesItem extends TermItem {
         return ImmutableList.copyOf(uniqueTerms);
     }
 
-    private void setMaxIndex() {
-        int maxIndex = 0;
-        int currentIndex = 0;
-        double maxScore = 0.0d;
-        boolean first = true;
-        for (Alternative val : this.alternatives) {
-            if (first) {
-                first = false;
-                maxIndex = 0;
-                maxScore = val.exactness;
-            } else {
-                if (val.exactness > maxScore) {
-                    maxScore = val.exactness;
-                    maxIndex = currentIndex;
-                }
-            }
-            ++currentIndex;
-        }
-        this.maxIndex = maxIndex;
-    }
-
     @Override
     public String stringValue() {
         StringBuilder builder = new StringBuilder();
diff --git a/container-search/src/main/java/com/yahoo/search/query/parser/ParserEnvironment.java b/container-search/src/main/java/com/yahoo/search/query/parser/ParserEnvironment.java
index 9e53f9d8ea9..94b9bf6ce65 100644
--- a/container-search/src/main/java/com/yahoo/search/query/parser/ParserEnvironment.java
+++ b/container-search/src/main/java/com/yahoo/search/query/parser/ParserEnvironment.java
@@ -4,7 +4,6 @@ package com.yahoo.search.query.parser;
 import com.yahoo.language.Linguistics;
 import com.yahoo.language.simple.SimpleLinguistics;
 import com.yahoo.prelude.IndexFacts;
-import com.yahoo.prelude.query.parser.SpecialTokenRegistry;
 import com.yahoo.prelude.query.parser.SpecialTokens;
 import com.yahoo.search.Searcher;
 import com.yahoo.search.searchchain.Execution;