You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by rm...@apache.org on 2012/01/26 19:27:52 UTC

svn commit: r1236342 - in /lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src: java/org/apache/lucene/analysis/phonetic/BeiderMorseFilter.java test/org/apache/lucene/analysis/phonetic/TestBeiderMorseFilter.java

Author: rmuir
Date: Thu Jan 26 18:27:52 2012
New Revision: 1236342

URL: http://svn.apache.org/viewvc?rev=1236342&view=rev
Log:
LUCENE-3720: add warning+experimental and disable test

Modified:
    lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/java/org/apache/lucene/analysis/phonetic/BeiderMorseFilter.java
    lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/test/org/apache/lucene/analysis/phonetic/TestBeiderMorseFilter.java

Modified: lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/java/org/apache/lucene/analysis/phonetic/BeiderMorseFilter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/java/org/apache/lucene/analysis/phonetic/BeiderMorseFilter.java?rev=1236342&r1=1236341&r2=1236342&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/java/org/apache/lucene/analysis/phonetic/BeiderMorseFilter.java (original)
+++ lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/java/org/apache/lucene/analysis/phonetic/BeiderMorseFilter.java Thu Jan 26 18:27:52 2012
@@ -32,7 +32,14 @@ import org.apache.lucene.analysis.tokena
 
 /**
  * TokenFilter for Beider-Morse phonetic encoding.
+ * <p>
+ * <b><font color="red">
+ * WARNING: some inputs can cause extremely high RAM usage! 
+ * https://issues.apache.org/jira/browse/CODEC-132
+ * </font></b>
+ * </p>
  * @see BeiderMorseEncoder
+ * @lucene.experimental
  */
 public final class BeiderMorseFilter extends TokenFilter {
   private final PhoneticEngine engine;

Modified: lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/test/org/apache/lucene/analysis/phonetic/TestBeiderMorseFilter.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/test/org/apache/lucene/analysis/phonetic/TestBeiderMorseFilter.java?rev=1236342&r1=1236341&r2=1236342&view=diff
==============================================================================
--- lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/test/org/apache/lucene/analysis/phonetic/TestBeiderMorseFilter.java (original)
+++ lucene/dev/branches/branch_3x/lucene/contrib/analyzers/phonetic/src/test/org/apache/lucene/analysis/phonetic/TestBeiderMorseFilter.java Thu Jan 26 18:27:52 2012
@@ -29,6 +29,7 @@ import org.apache.lucene.analysis.BaseTo
 import org.apache.lucene.analysis.MockTokenizer;
 import org.apache.lucene.analysis.ReusableAnalyzerBase;
 import org.apache.lucene.analysis.Tokenizer;
+import org.junit.Ignore;
 
 /** Tests {@link BeiderMorseFilter} */
 public class TestBeiderMorseFilter extends BaseTokenStreamTestCase {
@@ -87,6 +88,7 @@ public class TestBeiderMorseFilter exten
         new int[] { 1 });
   }
   
+  @Ignore("broken: causes OOM on some strings (https://issues.apache.org/jira/browse/CODEC-132)")
   public void testRandom() throws Exception {
     checkRandomData(random, analyzer, 1000 * RANDOM_MULTIPLIER); 
   }