You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ja...@apache.org on 2015/02/12 11:18:33 UTC

svn commit: r1659197 - in /lucene/dev/trunk: lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FreeTextSuggester.java solr/CHANGES.txt

Author: janhoy
Date: Thu Feb 12 10:18:33 2015
New Revision: 1659197

URL: http://svn.apache.org/r1659197
Log:
SOLR-7084: FreeTextSuggester Nullpointer when building dictionary

Modified:
    lucene/dev/trunk/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FreeTextSuggester.java
    lucene/dev/trunk/solr/CHANGES.txt

Modified: lucene/dev/trunk/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FreeTextSuggester.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FreeTextSuggester.java?rev=1659197&r1=1659196&r2=1659197&view=diff
==============================================================================
--- lucene/dev/trunk/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FreeTextSuggester.java (original)
+++ lucene/dev/trunk/lucene/suggest/src/java/org/apache/lucene/search/suggest/analyzing/FreeTextSuggester.java Thu Feb 12 10:18:33 2015
@@ -470,6 +470,9 @@ public class FreeTextSuggester extends L
     if (contexts != null) {
       throw new IllegalArgumentException("this suggester doesn't support contexts");
     }
+    if (fst == null) {
+      throw new IllegalStateException("Lookup not supported at this time");
+    }
 
     try (TokenStream ts = queryAnalyzer.tokenStream("", key.toString())) {
       TermToBytesRefAttribute termBytesAtt = ts.addAttribute(TermToBytesRefAttribute.class);

Modified: lucene/dev/trunk/solr/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/CHANGES.txt?rev=1659197&r1=1659196&r2=1659197&view=diff
==============================================================================
--- lucene/dev/trunk/solr/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/CHANGES.txt Thu Feb 12 10:18:33 2015
@@ -131,6 +131,9 @@ Bug Fixes
 * SOLR-6214: Snapshots numberToKeep param only keeps n-1 backups.
   (Mathias H., Ramana, Varun Thacker via shalin)
 
+* SOLR-7084: FreeTextSuggester: Better error message when doing a lookup
+  during dictionary build. Used to be nullpointer (janhoy)
+
 Optimizations
 ----------------------
  * SOLR-7049: Move work done by the LIST Collections API call to the Collections