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 2009/01/09 18:48:16 UTC
svn commit: r733097 -
/lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/impl/StreamingUpdateSolrServer.java
Author: ryan
Date: Fri Jan 9 09:48:16 2009
New Revision: 733097
URL: http://svn.apache.org/viewvc?rev=733097&view=rev
Log:
SOLR-906 -- don't recursivly call run() -- if the server is getting lots of errors, this could be called often and result in a stack overflow error
Modified:
lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/impl/StreamingUpdateSolrServer.java
Modified: lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/impl/StreamingUpdateSolrServer.java
URL: http://svn.apache.org/viewvc/lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/impl/StreamingUpdateSolrServer.java?rev=733097&r1=733096&r2=733097&view=diff
==============================================================================
--- lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/impl/StreamingUpdateSolrServer.java (original)
+++ lucene/solr/trunk/src/solrj/org/apache/solr/client/solrj/impl/StreamingUpdateSolrServer.java Fri Jan 9 09:48:16 2009
@@ -156,9 +156,6 @@
method.releaseConnection();
}
catch( Exception ex ){}
- if( !queue.isEmpty() ) {
- run(); // run again, just in case
- }
// remove it from the list of running things...
synchronized (runners) {