You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kylin.apache.org by xx...@apache.org on 2022/08/26 10:37:08 UTC

[kylin] branch kylin5 updated: KYLIN-5240 fix sort by duration in jobs list

This is an automated email from the ASF dual-hosted git repository.

xxyu pushed a commit to branch kylin5
in repository https://gitbox.apache.org/repos/asf/kylin.git


The following commit(s) were added to refs/heads/kylin5 by this push:
     new 0a85d6f528 KYLIN-5240 fix sort by duration in jobs list
0a85d6f528 is described below

commit 0a85d6f528f0a75e5d363f2166ac9363982cbc2e
Author: Mukvin <bo...@163.com>
AuthorDate: Fri Aug 26 16:34:43 2022 +0800

    KYLIN-5240 fix sort by duration in jobs list
---
 .../src/main/java/org/apache/kylin/rest/service/JobService.java     | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

diff --git a/src/data-loading-service/src/main/java/org/apache/kylin/rest/service/JobService.java b/src/data-loading-service/src/main/java/org/apache/kylin/rest/service/JobService.java
index c648fdec8d..0182d69e74 100644
--- a/src/data-loading-service/src/main/java/org/apache/kylin/rest/service/JobService.java
+++ b/src/data-loading-service/src/main/java/org/apache/kylin/rest/service/JobService.java
@@ -145,7 +145,7 @@ public class JobService extends BasicService implements JobSupporter {
 
     private static final Map<String, String> jobTypeMap = Maps.newHashMap();
 
-    private static final String TOTAL_DURATION = "total_duration";
+    private static final String DURATION = "duration";
     private static final String LAST_MODIFIED = "last_modified";
     public static final String EXCEPTION_CODE_PATH = "exception_to_code.json";
     public static final String EXCEPTION_CODE_DEFAULT = "KE-030001000";
@@ -233,8 +233,8 @@ public class JobService extends BasicService implements JobSupporter {
 
     private List<ExecutableResponse> sortTotalDurationList(List<ExecutableResponse> result, final JobFilter jobFilter) {
         //constructing objects takes time
-        if (StringUtils.isNotEmpty(jobFilter.getSortBy()) && jobFilter.getSortBy().equals(TOTAL_DURATION)) {
-            Collections.sort(result, propertyComparator(TOTAL_DURATION, !jobFilter.isReverse()));
+        if (StringUtils.isNotEmpty(jobFilter.getSortBy()) && jobFilter.getSortBy().equals(DURATION)) {
+            Collections.sort(result, propertyComparator(DURATION, !jobFilter.isReverse()));
         }
         return result;
     }