kylin-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From liy...@apache.org
Subject [05/12] kylin git commit: minor, output error message to response of diagnosis
Date Mon, 10 Oct 2016 13:54:19 GMT
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-hbase1.x
Commit: a2fb71d76aa342c7917a508ec981d232fb818f5a
Parents: 7ec0aac
Author: lidongsjtu <lidong@apache.org>
Authored: Sun Oct 9 12:50:33 2016 +0800
Committer: lidongsjtu <lidong@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);
         }
     }
 }


Mime
View raw message