KYLIN-2940 List job restful throw NPE when time filter not set

Signed-off-by: shaofengshi <shaofeng...@apache.org>


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

Branch: refs/heads/master
Commit: b8f2292641ae6d00b207196f6294de36e2a17bd6
Parents: e0d8dda
Author: sanjulian <julian....@hotmail.com>
Authored: Thu Dec 7 14:48:02 2017 +0800
Committer: shaofengshi <shaofeng...@apache.org>
Committed: Fri Dec 8 18:57:29 2017 +0800

----------------------------------------------------------------------
 .../java/org/apache/kylin/rest/controller/JobController.java    | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)
----------------------------------------------------------------------


http://git-wip-us.apache.org/repos/asf/kylin/blob/b8f22926/server-base/src/main/java/org/apache/kylin/rest/controller/JobController.java
----------------------------------------------------------------------
diff --git 
a/server-base/src/main/java/org/apache/kylin/rest/controller/JobController.java 
b/server-base/src/main/java/org/apache/kylin/rest/controller/JobController.java
index 749c872..ca3be99 100644
--- 
a/server-base/src/main/java/org/apache/kylin/rest/controller/JobController.java
+++ 
b/server-base/src/main/java/org/apache/kylin/rest/controller/JobController.java
@@ -69,7 +69,10 @@ public class JobController extends BasicController {
             }
         }
 
-        JobTimeFilterEnum timeFilter = 
JobTimeFilterEnum.getByCode(jobRequest.getTimeFilter());
+        JobTimeFilterEnum timeFilter = JobTimeFilterEnum.LAST_ONE_WEEK;
+        if (null != jobRequest.getTimeFilter()) {
+            timeFilter = 
JobTimeFilterEnum.getByCode(jobRequest.getTimeFilter());
+        }
 
         try {
             jobInstanceList = jobService.searchJobs(jobRequest.getCubeName(), 
jobRequest.getProjectName(), statusList,

Reply via email to