You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by mi...@apache.org on 2012/11/03 16:20:47 UTC

svn commit: r1405356 - in /lucene/dev/branches/branch_4x: ./ lucene/ lucene/suggest/ lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FuzzySuggester.java

Author: mikemccand
Date: Sat Nov  3 15:20:46 2012
New Revision: 1405356

URL: http://svn.apache.org/viewvc?rev=1405356&view=rev
Log:
remove invalid comment: I hadn't run the right perf test

Modified:
    lucene/dev/branches/branch_4x/   (props changed)
    lucene/dev/branches/branch_4x/lucene/   (props changed)
    lucene/dev/branches/branch_4x/lucene/suggest/   (props changed)
    lucene/dev/branches/branch_4x/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FuzzySuggester.java

Modified: lucene/dev/branches/branch_4x/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FuzzySuggester.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_4x/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FuzzySuggester.java?rev=1405356&r1=1405355&r2=1405356&view=diff
==============================================================================
--- lucene/dev/branches/branch_4x/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FuzzySuggester.java (original)
+++ lucene/dev/branches/branch_4x/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FuzzySuggester.java Sat Nov  3 15:20:46 2012
@@ -217,9 +217,6 @@ public final class FuzzySuggester extend
       // this only happens if you have multiple paths anyway (e.g. synonyms)
       BasicOperations.determinize(a);
 
-      // Does not seem to help (and hurt maybe a bit: 6-9
-      // prefix went from 19 to 18 kQPS):
-      // a.reduce();
       return a;
     }
   }