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/10 16:30:51 UTC

svn commit: r1229600 - /lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java

Author: markrmiller
Date: Tue Jan 10 15:30:50 2012
New Revision: 1229600

URL: http://svn.apache.org/viewvc?rev=1229600&view=rev
Log:
this has a test now

Modified:
    lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java

Modified: lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java
URL: http://svn.apache.org/viewvc/lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java?rev=1229600&r1=1229599&r2=1229600&view=diff
==============================================================================
--- lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java (original)
+++ lucene/dev/branches/solrcloud/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java Tue Jan 10 15:30:50 2012
@@ -241,7 +241,6 @@ public class DistributedUpdateProcessor 
   }
  
   // TODO: optionally fail if n replicas are not reached...
-  // nocommit: what the hell - doesnt seem to fail when cannot forward - need to check that...
   private void doFinish() {
     // TODO: if not a forward and replication req is not specified, we could
     // send in a background thread