when retrying, output the offending exception
Project: http://git-wip-us.apache.org/repos/asf/curator/repo Commit: http://git-wip-us.apache.org/repos/asf/curator/commit/4ce1f9d0 Tree: http://git-wip-us.apache.org/repos/asf/curator/tree/4ce1f9d0 Diff: http://git-wip-us.apache.org/repos/asf/curator/diff/4ce1f9d0 Branch: refs/heads/master Commit: 4ce1f9d018227a03523e0e973cb1571a17a5ee5d Parents: 4bc6063 Author: randgalt <[email protected]> Authored: Sat Jul 22 10:47:36 2017 -0500 Committer: randgalt <[email protected]> Committed: Sat Jul 22 10:47:36 2017 -0500 ---------------------------------------------------------------------- .../java/org/apache/curator/test/BaseClassForTests.java | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) ---------------------------------------------------------------------- http://git-wip-us.apache.org/repos/asf/curator/blob/4ce1f9d0/curator-test/src/main/java/org/apache/curator/test/BaseClassForTests.java ---------------------------------------------------------------------- diff --git a/curator-test/src/main/java/org/apache/curator/test/BaseClassForTests.java b/curator-test/src/main/java/org/apache/curator/test/BaseClassForTests.java index 8238c82..1c87be4 100644 --- a/curator-test/src/main/java/org/apache/curator/test/BaseClassForTests.java +++ b/curator-test/src/main/java/org/apache/curator/test/BaseClassForTests.java @@ -41,7 +41,6 @@ public class BaseClassForTests protected TestingServer server; private final Logger log = LoggerFactory.getLogger(getClass()); - private static final int RETRY_WAIT_MS = 5000; private static final String INTERNAL_PROPERTY_DONT_LOG_CONNECTION_ISSUES; private static final String INTERNAL_PROPERTY_REMOVE_WATCHERS_IN_FOREGROUND; private static final String INTERNAL_PROPERTY_VALIDATE_NAMESPACE_WATCHER_MAP_EMPTY; @@ -167,7 +166,14 @@ public class BaseClassForTests return false; } - log.error("Retrying 1 time"); + if ( result.getThrowable() != null ) + { + log.error("Retrying 1 time", result.getThrowable()); + } + else + { + log.error("Retrying 1 time"); + } retryContext.isRetrying.set(true); return true; }
