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/12/13 10:25:30 UTC
[kylin] 23/25: KYLIN-5279 Modify the default value of null collation to low
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 66057d5c707e58e1b1be405121561ce779780008
Author: sibing.zhang <19...@qq.com>
AuthorDate: Fri Dec 9 10:57:19 2022 +0800
KYLIN-5279 Modify the default value of null collation to low
---
.../apache/kylin/query/engine/KECalciteConfig.java | 22 +++++++++++-----------
1 file changed, 11 insertions(+), 11 deletions(-)
diff --git a/src/query-common/src/main/java/org/apache/kylin/query/engine/KECalciteConfig.java b/src/query-common/src/main/java/org/apache/kylin/query/engine/KECalciteConfig.java
index 148c4f5dcd..f4a35baf96 100644
--- a/src/query-common/src/main/java/org/apache/kylin/query/engine/KECalciteConfig.java
+++ b/src/query-common/src/main/java/org/apache/kylin/query/engine/KECalciteConfig.java
@@ -55,19 +55,19 @@ public class KECalciteConfig extends CalciteConnectionConfigImpl {
public NullCollation defaultNullCollation() {
String nullCollation = kylinConfig.getCalciteExtrasProperties().get("defaultNullCollation");
if (null == nullCollation) {
- return NullCollation.HIGH;
+ return NullCollation.LOW;
}
switch (nullCollation.toLowerCase(Locale.ROOT)) {
- case "low":
- return NullCollation.LOW;
- case "high":
- return NullCollation.HIGH;
- case "first":
- return NullCollation.FIRST;
- case "last":
- return NullCollation.LAST;
- default:
- throw new UnsupportedOperationException("unsupported null collation type: " + nullCollation);
+ case "low":
+ return NullCollation.LOW;
+ case "high":
+ return NullCollation.HIGH;
+ case "first":
+ return NullCollation.FIRST;
+ case "last":
+ return NullCollation.LAST;
+ default:
+ throw new UnsupportedOperationException("unsupported null collation type: " + nullCollation);
}
}