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/02/20 02:45:09 UTC

svn commit: r1291115 - in /lucene/dev/trunk/solr/contrib/dataimporthandler: CHANGES.txt src/java/org/apache/solr/handler/dataimport/DataImporter.java

Author: rmuir
Date: Mon Feb 20 01:45:09 2012
New Revision: 1291115

URL: http://svn.apache.org/viewvc?rev=1291115&view=rev
Log:
SOLR-3142: don't auto-optimize in dataimports

Modified:
    lucene/dev/trunk/solr/contrib/dataimporthandler/CHANGES.txt
    lucene/dev/trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImporter.java

Modified: lucene/dev/trunk/solr/contrib/dataimporthandler/CHANGES.txt
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/contrib/dataimporthandler/CHANGES.txt?rev=1291115&r1=1291114&r2=1291115&view=diff
==============================================================================
--- lucene/dev/trunk/solr/contrib/dataimporthandler/CHANGES.txt (original)
+++ lucene/dev/trunk/solr/contrib/dataimporthandler/CHANGES.txt Mon Feb 20 01:45:09 2012
@@ -19,6 +19,12 @@ New Features
 * SOLR-1499: Added SolrEntityProcessor that imports data from another Solr core or instance based on a specified query.
              (Lance Norskog, Erik Hatcher, Pulkit Singhal, Ahmet Arslan, Luca Cavanna, Martijn van Groningen)
 
+Changes in Runtime Behavior
+----------------------
+* SOLR-3142: Imports no longer default optimize to true, instead false. If you want to force all segments to be merged
+             into one, you can specify this parameter yourself. NOTE: this can be very expensive operation and usually
+             does not make sense for delta-imports.  (Robert MUir)
+
 ==================  3.5.0 ==================
 
 Bug Fixes

Modified: lucene/dev/trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImporter.java
URL: http://svn.apache.org/viewvc/lucene/dev/trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImporter.java?rev=1291115&r1=1291114&r2=1291115&view=diff
==============================================================================
--- lucene/dev/trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImporter.java (original)
+++ lucene/dev/trunk/solr/contrib/dataimporthandler/src/java/org/apache/solr/handler/dataimport/DataImporter.java Mon Feb 20 01:45:09 2012
@@ -523,7 +523,7 @@ public class DataImporter {
 
     public boolean commit = true;
 
-    public boolean optimize = true;
+    public boolean optimize = false;
 
     public int start = 0;
 
@@ -571,7 +571,7 @@ public class DataImporter {
       if (requestParams.containsKey("clean"))
         clean = StrUtils.parseBool((String) requestParams.get("clean"),true);
       if (requestParams.containsKey("optimize")) {
-        optimize = StrUtils.parseBool((String) requestParams.get("optimize"),true);
+        optimize = StrUtils.parseBool((String) requestParams.get("optimize"),false);
         if (optimize)
           commit = true;
       }