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 2023/07/06 05:53:33 UTC
[kylin] 16/19: KYLIN-5608 add agg groups fix order by input params
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 <44...@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));