[kylin] 05/15: KYLIN-5351 minor fix of KYLIN-5333

[xxyu]

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

commit 21c31f4556853d16a8cb9336e7887f2afeaae14b
Author: Hang Jia <754332...@qq.com>
AuthorDate: Mon Oct 24 16:17:22 2022 +0800

    KYLIN-5351 minor fix of KYLIN-5333
    
    Co-authored-by: Hang Jia <754332...@qq.com>
---
 .../java/org/apache/kylin/rest/controller/v2/JobControllerV2.java     | 4 ++++
 1 file changed, 4 insertions(+)

diff --git 
a/src/data-loading-server/src/main/java/org/apache/kylin/rest/controller/v2/JobControllerV2.java
 
b/src/data-loading-server/src/main/java/org/apache/kylin/rest/controller/v2/JobControllerV2.java
index 414563adf8..fae3e21e61 100644
--- 
a/src/data-loading-server/src/main/java/org/apache/kylin/rest/controller/v2/JobControllerV2.java
+++ 
b/src/data-loading-server/src/main/java/org/apache/kylin/rest/controller/v2/JobControllerV2.java
@@ -90,6 +90,10 @@ public class JobControllerV2 extends BaseController {
             @RequestParam(value = "sortBy", required = false, defaultValue = 
"last_modified") String sortBy,
             @RequestParam(value = "sortby", required = false) String sortby, 
//param for 3x
             @RequestParam(value = "reverse", required = false, defaultValue = 
"true") Boolean reverse) {
+        // 3x default last_modify
+        if (!StringUtils.isEmpty(sortby) && !"last_modify".equals(sortby)) {
+            sortBy = sortby;
+        }
         checkNonNegativeIntegerArg("pageOffset", pageOffset);
         checkNonNegativeIntegerArg("pageSize", pageSize);
         List<String> statuses = Lists.newArrayList();