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 e594cf3a3e2625f5c58dacb2b41fa912e5e13b21
Author: xiang.li <447399...@qq.com>
AuthorDate: Fri Apr 14 15:36:55 2023 +0800

    KYLIN-5608 add agg groups fix order by input params
---
 .../main/java/org/apache/kylin/rest/service/FusionIndexService.java   | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git 
a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/FusionIndexService.java
 
b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/FusionIndexService.java
index 71cece5de9..2d5c51c47d 100644
--- 
a/src/modeling-service/src/main/java/org/apache/kylin/rest/service/FusionIndexService.java
+++ 
b/src/modeling-service/src/main/java/org/apache/kylin/rest/service/FusionIndexService.java
@@ -25,11 +25,11 @@ import java.util.Arrays;
 import java.util.HashMap;
 import java.util.HashSet;
 import java.util.LinkedHashMap;
+import java.util.LinkedHashSet;
 import java.util.List;
 import java.util.Locale;
 import java.util.Map;
 import java.util.Set;
-import java.util.TreeSet;
 import java.util.function.Function;
 import java.util.stream.Collectors;
 
@@ -506,7 +506,7 @@ public class FusionIndexService extends BasicService {
         }
         Set<String> set = Arrays.stream(dimOrMeaNames)
                 .map(str -> aggGroupParams == AggGroupParams.MEASURE ? str : 
StringUtils.upperCase(str, Locale.ROOT))
-                .collect(Collectors.toCollection(TreeSet::new));
+                .collect(Collectors.toCollection(LinkedHashSet::new));
         if (set.size() < dimOrMeaNames.length) {
             throw new IllegalStateException(
                     "Dimension or measure in agg group must not contain 
duplication: " + Arrays.asList(dimOrMeaNames));

Reply via email to