From f50980d6513c367657374988083e32039c454992 Mon Sep 17 00:00:00 2001 From: Mars Hall Date: Fri, 16 Jun 2017 11:45:03 -0700 Subject: [PATCH] Fix to show stacktrace for errors thrown in `queries.json` REST API --- .../org/apache/predictionio/workflow/CreateServer.scala | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/core/src/main/scala/org/apache/predictionio/workflow/CreateServer.scala b/core/src/main/scala/org/apache/predictionio/workflow/CreateServer.scala index 31b783134a..8ef36e26ef 100644 --- a/core/src/main/scala/org/apache/predictionio/workflow/CreateServer.scala +++ b/core/src/main/scala/org/apache/predictionio/workflow/CreateServer.scala @@ -594,14 +594,14 @@ class ServerActor[Q, P]( } } catch { case e: MappingException => - log.error( - s"Query '$queryString' is invalid. Reason: ${e.getMessage}") + val msg = s"Query:\n$queryString\n\nStack Trace:\n" + + s"${getStackTraceString(e)}\n\n" + log.error(msg) args.logUrl map { url => remoteLog( url, args.logPrefix.getOrElse(""), - s"Query:\n$queryString\n\nStack Trace:\n" + - s"${getStackTraceString(e)}\n\n") + msg) } complete(StatusCodes.BadRequest, e.getMessage) case e: Throwable =>