Duplicate log entry in TransportReplicationAction

jasontedor · jasontedor · commit 0aff8055e45d · 2016-02-09T06:45:50.000-05:00
This commit removes a logging statement from TransportReplicationAction
that occurs when handling a catastrophic non-channel failure while
failing a shard. The same information is also logged from
ShardStateAction when the master responds to the shard failure request
there.
diff --git a/core/src/main/java/org/elasticsearch/action/support/replication/TransportReplicationAction.java b/core/src/main/java/org/elasticsearch/action/support/replication/TransportReplicationAction.java
@@ -954,7 +954,6 @@ public void onSuccess() {
 
                                         @Override
                                         public void onFailure(Throwable shardFailedError) {
-                                            logger.error("[{}] catastrophic error while failing replica shard [{}] for [{}]", shardFailedError, shardId, shard, exp);
                                             if (shardFailedError instanceof ShardStateAction.NoLongerPrimaryShardException) {
                                                 // we are no longer the primary, fail ourselves and start over
                                                 ShardRouting primaryShard = indexShardReference.routingEntry();
