You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@lucene.apache.org by ma...@apache.org on 2012/01/02 21:10:39 UTC

svn commit: r1226533 - /lucene/dev/branches/solrcloud/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttpSolrServer.java

Author: markrmiller
Date: Mon Jan  2 20:10:39 2012
New Revision: 1226533

URL: http://svn.apache.org/viewvc?rev=1226533&view=rev
Log:
403 may be returned on shutdown

Modified:
    lucene/dev/branches/solrcloud/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttpSolrServer.java

Modified: lucene/dev/branches/solrcloud/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttpSolrServer.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/solrcloud/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttpSolrServer.java?rev=1226533&r1=1226532&r2=1226533&view=diff
==============================================================================
--- lucene/dev/branches/solrcloud/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttpSolrServer.java (original)
+++ lucene/dev/branches/solrcloud/solr/solrj/src/java/org/apache/solr/client/solrj/impl/LBHttpSolrServer.java Mon Jan  2 20:10:39 2012
@@ -250,8 +250,8 @@ public class LBHttpSolrServer extends So
         rsp.rsp = server.request(req.getRequest());
         return rsp; // SUCCESS
       } catch (SolrException e) {
-        // we retry on 404 - you can see this on solr shutdown
-        if (e.code() == 404) {
+        // we retry on 404 or 403 - you can see this on solr shutdown
+        if (e.code() == 404 || e.code() == 403) {
           ex = addZombie(server, e);
         } else {
           // Server is alive but the request was likely malformed or invalid