From 1cef531165e9488a9032596ae7b907d0e1fcc2c1 Mon Sep 17 00:00:00 2001 From: Chris Earle Date: Fri, 2 Jun 2017 10:01:21 -0400 Subject: [PATCH] Always Accumulate Transport Exceptions (#1619) This is the x-pack side of the removal of `accumulateExceptions()` for both `TransportNodesAction` and `TransportTasksAction`. There are occasional, random failures that occur during API calls that are silently ignored from the caller's perspective, which also leads to weird API responses that have no response and also no errors, which is obviously untrue. --- .../elasticsearch/persistent/TestPersistentTasksPlugin.java | 4 ---- 1 file changed, 4 deletions(-) diff --git a/server/src/test/java/org/elasticsearch/persistent/TestPersistentTasksPlugin.java b/server/src/test/java/org/elasticsearch/persistent/TestPersistentTasksPlugin.java index 5f3b6c13aaa..95917f205c2 100644 --- a/server/src/test/java/org/elasticsearch/persistent/TestPersistentTasksPlugin.java +++ b/server/src/test/java/org/elasticsearch/persistent/TestPersistentTasksPlugin.java @@ -546,10 +546,6 @@ public class TestPersistentTasksPlugin extends Plugin implements ActionPlugin { listener.onResponse(new TestTaskResponse()); } - @Override - protected boolean accumulateExceptions() { - return false; - } }