You are viewing a plain text version of this content. The canonical link for it is here.
Posted to solr-commits@lucene.apache.org by ry...@apache.org on 2007/10/13 01:27:20 UTC

svn commit: r584319 - in /lucene/solr/trunk: CHANGES.txt src/java/org/apache/solr/analysis/BaseTokenizerFactory.java src/java/org/apache/solr/schema/IndexSchema.java

Author: ryan
Date: Fri Oct 12 16:27:12 2007
New Revision: 584319

URL: http://svn.apache.org/viewvc?rev=584319&view=rev
Log:
SOLR-215 -- adding CHANGES.txt and moving the deprecation warning from BaseXXXFactory to IndexSchema.java

Modified:
    lucene/solr/trunk/CHANGES.txt
    lucene/solr/trunk/src/java/org/apache/solr/analysis/BaseTokenizerFactory.java
    lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java

Modified: lucene/solr/trunk/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/CHANGES.txt?rev=584319&r1=584318&r2=584319&view=diff
==============================================================================
--- lucene/solr/trunk/CHANGES.txt (original)
+++ lucene/solr/trunk/CHANGES.txt Fri Oct 12 16:27:12 2007
@@ -197,6 +197,16 @@
 
  3. Upgraded to Lucene 2.2.0; June 18, 2007.
 
+ 4. SOLR-215: In an push to support multiple SolrCores, the TokenizerFactory
+    and TokenFilterFactory initalization interface has changed to accept the 
+    solrConfig.  Initialization should happen in:
+ 		  init(SolrConfig solrConfig, Map<String,String> args)
+    rather then:
+      init(Map<String,String> args)
+    Existing classes should continue to work, but it is encouraged to update
+    the initialization code.
+ 
+
 ================== Release 1.2, 20070602 ==================
 
 Upgrading from Solr 1.1

Modified: lucene/solr/trunk/src/java/org/apache/solr/analysis/BaseTokenizerFactory.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/analysis/BaseTokenizerFactory.java?rev=584319&r1=584318&r2=584319&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/analysis/BaseTokenizerFactory.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/analysis/BaseTokenizerFactory.java Fri Oct 12 16:27:12 2007
@@ -35,7 +35,6 @@
   
   @Deprecated
   public void init(Map<String,String> args) {
-    //log.warning("calling the deprecated form of init; should be calling init(SolrConfig solrConfig, Map<String,String> args)");
     this.args=args;
   }
   

Modified: lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java?rev=584319&r1=584318&r2=584319&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/schema/IndexSchema.java Fri Oct 12 16:27:12 2007
@@ -629,10 +629,13 @@
     NamedNodeMap attrs = node.getAttributes();
     String className = DOMUtil.getAttr(attrs,"class","tokenizer");
     TokenizerFactory tfac = (TokenizerFactory)solrConfig.newInstance(className);
-    if (tfac instanceof SolrConfig.Initializable)
+    if (tfac instanceof SolrConfig.Initializable) {
       ((SolrConfig.Initializable)tfac).init(solrConfig, DOMUtil.toMapExcept(attrs,"class"));
-    else
+    }
+    else {
+      log.warning("calling the deprecated form of init; should be calling init(SolrConfig solrConfig, Map<String,String> args) " + className );
       tfac.init(DOMUtil.toMapExcept(attrs,"class"));
+    }
     
     return tfac;
   }
@@ -643,11 +646,13 @@
     NamedNodeMap attrs = node.getAttributes();
     String className = DOMUtil.getAttr(attrs,"class","token filter");
     TokenFilterFactory tfac = (TokenFilterFactory)solrConfig.newInstance(className);
-    if (tfac instanceof SolrConfig.Initializable)
+    if (tfac instanceof SolrConfig.Initializable) {
       ((SolrConfig.Initializable)tfac).init(solrConfig, DOMUtil.toMapExcept(attrs,"class"));
-    else
+    }
+    else {
+      log.warning("calling the deprecated form of init; should be calling init(SolrConfig solrConfig, Map<String,String> args) " + className );
       tfac.init(DOMUtil.toMapExcept(attrs,"class"));
-    
+    }
     return tfac;
   }