From f35c455d5b404a370522e8c990666e0f1f2c705f Mon Sep 17 00:00:00 2001 From: "Colin P. Mccabe" Date: Thu, 28 Mar 2019 11:03:46 -0700 Subject: [PATCH] MINOR: WorkerUtils#abort: fix bug in abort logic --- .../java/org/apache/kafka/trogdor/common/WorkerUtils.java | 8 ++++++-- 1 file changed, 6 insertions(+), 2 deletions(-) diff --git a/tools/src/main/java/org/apache/kafka/trogdor/common/WorkerUtils.java b/tools/src/main/java/org/apache/kafka/trogdor/common/WorkerUtils.java index ef6e275094e30..ad54c06c098a3 100644 --- a/tools/src/main/java/org/apache/kafka/trogdor/common/WorkerUtils.java +++ b/tools/src/main/java/org/apache/kafka/trogdor/common/WorkerUtils.java @@ -62,8 +62,12 @@ public final class WorkerUtils { */ public static void abort(Logger log, String what, Throwable exception, KafkaFutureImpl doneFuture) throws KafkaException { - log.warn("{} caught an exception: ", what, exception); - doneFuture.complete(exception.getMessage()); + log.warn("{} caught an exception", what, exception); + if (exception.getMessage() == null || exception.getMessage().isEmpty()) { + doneFuture.complete(exception.getClass().getCanonicalName()); + } else { + doneFuture.complete(exception.getMessage()); + } throw new KafkaException(exception); }