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));