You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@gobblin.apache.org by le...@apache.org on 2021/02/23 00:50:08 UTC

[gobblin] branch master updated: [GOBBLIN-1394] add stacktrace to error log callback dispatcher

This is an automated email from the ASF dual-hosted git repository.

lesun pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/gobblin.git


The following commit(s) were added to refs/heads/master by this push:
     new bdc6e01  [GOBBLIN-1394] add stacktrace to error log callback dispatcher
bdc6e01 is described below

commit bdc6e01f62228b6c8b7fcb8210eb45b6b9d4b169
Author: William Lo <lo...@gmail.com>
AuthorDate: Mon Feb 22 16:49:59 2021 -0800

    [GOBBLIN-1394] add stacktrace to error log callback dispatcher
    
    Closes #3233 from Will-Lo/add-error-log-callback
---
 .../java/org/apache/gobblin/util/callbacks/CallbacksDispatcher.java     | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/gobblin-utility/src/main/java/org/apache/gobblin/util/callbacks/CallbacksDispatcher.java b/gobblin-utility/src/main/java/org/apache/gobblin/util/callbacks/CallbacksDispatcher.java
index 144556c..9942406 100644
--- a/gobblin-utility/src/main/java/org/apache/gobblin/util/callbacks/CallbacksDispatcher.java
+++ b/gobblin-utility/src/main/java/org/apache/gobblin/util/callbacks/CallbacksDispatcher.java
@@ -171,7 +171,7 @@ public class CallbacksDispatcher<L> implements Closeable {
         res.cancellations.put(listener, cr);
       }
       else if (cr.hasFailed()) {
-        _log.error("Callback error: " + callbacks.get(i) + " on " + listener + ":" + cr.getError());
+        _log.error("Callback error: " + callbacks.get(i) + " on " + listener + ":" + cr.getError(), cr.getError());
         res.failures.put(listener, cr);
       }
       else {