You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by li...@apache.org on 2016/10/09 04:52:12 UTC

kylin git commit: minor, output error message to response of diagnosis

Repository: kylin
Updated Branches:
  refs/heads/yang21 7ec0aacc2 -> a2fb71d76


minor, output error message to response of diagnosis


Project: http://git-wip-us.apache.org/repos/asf/kylin/repo
Commit: http://git-wip-us.apache.org/repos/asf/kylin/commit/a2fb71d7
Tree: http://git-wip-us.apache.org/repos/asf/kylin/tree/a2fb71d7
Diff: http://git-wip-us.apache.org/repos/asf/kylin/diff/a2fb71d7

Branch: refs/heads/yang21
Commit: a2fb71d76aa342c7917a508ec981d232fb818f5a
Parents: 7ec0aac
Author: lidongsjtu <li...@apache.org>
Authored: Sun Oct 9 12:50:33 2016 +0800
Committer: lidongsjtu <li...@apache.org>
Committed: Sun Oct 9 12:50:33 2016 +0800

----------------------------------------------------------------------
 .../org/apache/kylin/rest/controller/DiagnosisController.java  | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/a2fb71d7/server-base/src/main/java/org/apache/kylin/rest/controller/DiagnosisController.java
----------------------------------------------------------------------
diff --git a/server-base/src/main/java/org/apache/kylin/rest/controller/DiagnosisController.java b/server-base/src/main/java/org/apache/kylin/rest/controller/DiagnosisController.java
index 826f888..d16547e 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/controller/DiagnosisController.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/controller/DiagnosisController.java
@@ -82,7 +82,7 @@ public class DiagnosisController extends BasicController {
         try {
             filePath = dgService.dumpProjectDiagnosisInfo(project);
         } catch (IOException e) {
-            throw new InternalErrorException("Failed to dump diagnosis info.", e);
+            throw new InternalErrorException("Failed to dump project diagnosis info. " + e.getMessage(), e);
         }
 
         setDownloadResponse(filePath, response);
@@ -98,7 +98,7 @@ public class DiagnosisController extends BasicController {
         try {
             filePath = dgService.dumpJobDiagnosisInfo(jobId);
         } catch (IOException e) {
-            throw new InternalErrorException("Failed to dump diagnosis info.", e);
+            throw new InternalErrorException("Failed to dump job diagnosis info. " + e.getMessage(), e);
         }
 
         setDownloadResponse(filePath, response);
@@ -114,7 +114,7 @@ public class DiagnosisController extends BasicController {
             IOUtils.copyLarge(fileInputStream, output);
             output.flush();
         } catch (IOException e) {
-            throw new InternalErrorException("Failed to dump diagnosis info.", e);
+            throw new InternalErrorException("Failed to create download for diagnosis. " + e.getMessage(), e);
         }
     }
 }