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 2017/06/05 05:23:23 UTC

[35/67] [abbrv] kylin git commit: minor, fix job controller

minor, fix job controller


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

Branch: refs/heads/master
Commit: c341a627de025dc4aa4c20a03c4337f685b476c1
Parents: e3a79c8
Author: Roger Shi <ro...@hotmail.com>
Authored: Sat May 27 18:34:08 2017 +0800
Committer: 成 <ch...@kyligence.io>
Committed: Sat May 27 18:36:33 2017 +0800

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


http://git-wip-us.apache.org/repos/asf/kylin/blob/c341a627/server-base/src/main/java/org/apache/kylin/rest/controller2/JobControllerV2.java
----------------------------------------------------------------------
diff --git a/server-base/src/main/java/org/apache/kylin/rest/controller2/JobControllerV2.java b/server-base/src/main/java/org/apache/kylin/rest/controller2/JobControllerV2.java
index abc8621..2bcc11b 100644
--- a/server-base/src/main/java/org/apache/kylin/rest/controller2/JobControllerV2.java
+++ b/server-base/src/main/java/org/apache/kylin/rest/controller2/JobControllerV2.java
@@ -131,18 +131,21 @@ public class JobControllerV2 extends BasicController {
         if (sortby.equals("last_modify")) {
             if (reverse) {
                 Collections.sort(jobInstanceList, lastModifyComparatorReverse);
+            } else {
+                Collections.sort(jobInstanceList, lastModifyComparator);
             }
-            Collections.sort(jobInstanceList, lastModifyComparator);
         } else if (sortby.equals("job_name")) {
             if (reverse) {
                 Collections.sort(jobInstanceList, jobNameComparatorReverse);
+            } else {
+                Collections.sort(jobInstanceList, jobNameComparator);
             }
-            Collections.sort(jobInstanceList, jobNameComparator);
         } else if (sortby.equals("cube_name")) {
             if (reverse) {
                 Collections.sort(jobInstanceList, cubeNameComparatorReverse);
+            } else {
+                Collections.sort(jobInstanceList, cubeNameComparator);
             }
-            Collections.sort(jobInstanceList, cubeNameComparator);
         }
 
         int offset = pageOffset * pageSize;