svn commit: r711574 - /lucene/solr/trunk/src/java/org/apache/solr/handler/component/SearchHandler.java

[yo...@apache.org]

Author: yonik
Date: Wed Nov  5 06:32:55 2008
New Revision: 711574

URL: http://svn.apache.org/viewvc?rev=711574&view=rev
Log:
remove the now redundant setResponseParser

Modified:
    lucene/solr/trunk/src/java/org/apache/solr/handler/component/SearchHandler.java

Modified: lucene/solr/trunk/src/java/org/apache/solr/handler/component/SearchHandler.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/java/org/apache/solr/handler/component/SearchHandler.java?rev=711574&r1=711573&r2=711574&view=diff
==============================================================================
--- lucene/solr/trunk/src/java/org/apache/solr/handler/component/SearchHandler.java (original)
+++ lucene/solr/trunk/src/java/org/apache/solr/handler/component/SearchHandler.java Wed Nov  5 06:32:55 2008
@@ -379,7 +379,7 @@
           // String url = "http://" + shard + "/select";
           String url = "http://" + shard;
 
-          params.remove(CommonParams.WT); // use default (or should we explicitly set it?)
+          params.remove(CommonParams.WT); // use default (currently javabin)
           params.remove(CommonParams.VERSION);
 
           SolrServer server = new CommonsHttpSolrServer(url, client);
@@ -387,7 +387,9 @@
           // use generic request to avoid extra processing of queries
           QueryRequest req = new QueryRequest(params);
           req.setMethod(SolrRequest.METHOD.POST);
-          req.setResponseParser(new BinaryResponseParser());  // this sets the wt param
+
+          // no need to set the response parser as binary is the default
+          // req.setResponseParser(new BinaryResponseParser());
           // srsp.rsp = server.request(req);
           // srsp.rsp = server.query(sreq.params);