You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@openwhisk.apache.org by mh...@apache.org on 2019/10/01 13:31:19 UTC

[openwhisk] branch master updated: Dump complete exception stacktrace upon failure in StoreUtil (#4653)

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

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


The following commit(s) were added to refs/heads/master by this push:
     new a29a941  Dump complete exception stacktrace upon failure in StoreUtil (#4653)
a29a941 is described below

commit a29a941dfcb552c12b98b11e2fd4ff543656d5fb
Author: Chetan Mehrotra <ch...@apache.org>
AuthorDate: Tue Oct 1 19:01:12 2019 +0530

    Dump complete exception stacktrace upon failure in StoreUtil (#4653)
---
 .../main/scala/org/apache/openwhisk/core/database/StoreUtils.scala | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/common/scala/src/main/scala/org/apache/openwhisk/core/database/StoreUtils.scala b/common/scala/src/main/scala/org/apache/openwhisk/core/database/StoreUtils.scala
index 1300a4c..e57b673 100644
--- a/common/scala/src/main/scala/org/apache/openwhisk/core/database/StoreUtils.scala
+++ b/common/scala/src/main/scala/org/apache/openwhisk/core/database/StoreUtils.scala
@@ -23,6 +23,7 @@ import akka.event.Logging.ErrorLevel
 import akka.stream.SinkShape
 import akka.stream.scaladsl.{Broadcast, Flow, GraphDSL, Keep, Sink}
 import akka.util.ByteString
+import com.google.common.base.Throwables
 import spray.json.DefaultJsonProtocol._
 import spray.json.{JsObject, JsValue, RootJsonFormat}
 import org.apache.openwhisk.common.{Logging, StartMarker, TransactionId}
@@ -41,7 +42,11 @@ private[database] object StoreUtils {
     f.failed.foreach {
       case _: ArtifactStoreException => // These failures are intentional and shouldn't trigger the catcher.
       case x =>
-        transid.failed(this, start, s"${failureMessage(x)} [${x.getClass.getSimpleName}]", ErrorLevel)
+        transid.failed(
+          this,
+          start,
+          s"${failureMessage(x)} [${x.getClass.getSimpleName}]\n" + Throwables.getStackTraceAsString(x),
+          ErrorLevel)
     }
     f
   }