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 <1948879...@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); } }