Github user dsmiley commented on a diff in the pull request:

    https://github.com/apache/lucene-solr/pull/275#discussion_r154458145
  
    --- Diff: 
solr/core/src/java/org/apache/solr/parser/SolrQueryParserBase.java ---
    @@ -78,6 +81,39 @@
       static final int MOD_NOT     = 10;
       static final int MOD_REQ     = 11;
     
    +  protected SynonymQueryStyle synonymQueryStyle = AS_SAME_TERM;
    +
    +  /**
    +   *  Query strategy when analyzed query terms overlap the same position 
(ie synonyms)
    +   *  consider if pants and khakis are query time synonyms
    +   *
    +   *  <li>{@link #AS_SAME_TERM}</li>
    +   *  <li>{@link #PICK_BEST}</li>
    +   *  <li>{@link #AS_DISTINCT_TERMS}</li>
    +   */
    +  public static enum SynonymQueryStyle {
    --- End diff --
    
    I like the new name, and thanks for improving the javadocs.  BTW that "li" 
HTML list is missing the "<ul> wrapper.  Or better IMO is simply drop this 
list; it has no value I think.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to