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 yo...@apache.org on 2010/04/11 03:03:19 UTC

svn commit: r932821 - /lucene/solr/branches/cloud/src/java/org/apache/solr/handler/component/TermsComponent.java

Author: yonik
Date: Sun Apr 11 01:03:19 2010
New Revision: 932821

URL: http://svn.apache.org/viewvc?rev=932821&view=rev
Log:
terms component needs to set rb.isDistrib for distributed operation

Modified:
    lucene/solr/branches/cloud/src/java/org/apache/solr/handler/component/TermsComponent.java

Modified: lucene/solr/branches/cloud/src/java/org/apache/solr/handler/component/TermsComponent.java
URL: http://svn.apache.org/viewvc/lucene/solr/branches/cloud/src/java/org/apache/solr/handler/component/TermsComponent.java?rev=932821&r1=932820&r2=932821&view=diff
==============================================================================
--- lucene/solr/branches/cloud/src/java/org/apache/solr/handler/component/TermsComponent.java (original)
+++ lucene/solr/branches/cloud/src/java/org/apache/solr/handler/component/TermsComponent.java Sun Apr 11 01:03:19 2010
@@ -58,6 +58,7 @@ public class TermsComponent extends Sear
     // TODO: temporary... this should go in a different component.
     String shards = params.get(ShardParams.SHARDS);
     if (shards != null) {
+      rb.isDistrib = true;
       if (params.get(ShardParams.SHARDS_QT) == null) {
         throw new SolrException(SolrException.ErrorCode.BAD_REQUEST, "No shards.qt parameter specified");
       }
@@ -250,9 +251,6 @@ public class TermsComponent extends Sear
     // base shard request on original parameters
     sreq.params = new ModifiableSolrParams(params);
 
-    // don't pass through the shards param
-    sreq.params.remove(ShardParams.SHARDS);
-
     // remove any limits for shards, we want them to return all possible
     // responses
     // we want this so we can calculate the correct counts
@@ -262,11 +260,6 @@ public class TermsComponent extends Sear
     sreq.params.set(TermsParams.TERMS_LIMIT, -1);
     sreq.params.set(TermsParams.TERMS_SORT, TermsParams.TERMS_SORT_INDEX);
 
-    // TODO: is there a better way to handle this?
-    String qt = params.get(CommonParams.QT);
-    if (qt != null) {
-      sreq.params.add(CommonParams.QT, qt);
-    }
     return sreq;
   }