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

Reply via email to