From 0d760aeddbc95f2de977dcaed781547f2cbf715f Mon Sep 17 00:00:00 2001 From: Ishan Chattopadhyaya Date: Tue, 7 Feb 2017 04:22:35 -0800 Subject: [PATCH] SOLR-5944: Reverting the previous fix for SolrCmdDistributor --- .../update/processor/DistributedUpdateProcessor.java | 10 +--------- 1 file changed, 1 insertion(+), 9 deletions(-) diff --git a/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java b/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java index e36b7c9df93..8f5d90993d1 100644 --- a/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java +++ b/solr/core/src/java/org/apache/solr/update/processor/DistributedUpdateProcessor.java @@ -799,16 +799,8 @@ public class DistributedUpdateProcessor extends UpdateRequestProcessor { if (cmd.isInPlaceUpdate()) { params.set(DISTRIB_INPLACE_PREVVERSION, String.valueOf(cmd.prevVersion)); - // Use synchronous=true so that a new connection is used, instead - // of the update being streamed through an existing streaming client. - // When using a streaming client, the previous update - // and the current in-place update (that depends on the previous update), if reordered - // in the stream, can result in the current update being bottled up behind the previous - // update in the stream and can lead to degraded performance. - cmdDistrib.distribAdd(cmd, nodes, params, true, replicationTracker); - } else { - cmdDistrib.distribAdd(cmd, nodes, params, false, replicationTracker); } + cmdDistrib.distribAdd(cmd, nodes, params, false, replicationTracker); } // TODO: what to do when no idField?