KE-11534 [Follow up]Remove the if operator check

Kyligence · Mar 20, 2024 · 2d37796 · 2d37796
1 parent fe103e6
commit 2d37796
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java b/core/src/main/java/org/apache/calcite/sql/fun/SqlLibraryOperators.java
@@ -120,8 +120,11 @@ private SqlLibraryOperators() {
               }),
           SqlFunctionCategory.SYSTEM) {
         @Override public boolean validRexOperands(int count, Litmus litmus) {
+          // Calcite 1.30 changed the if operator to case when,
+          // which causes Kylin computable columns to be unusable
+          return true;
           // IF is translated to RexNode by expanding to CASE.
-          return litmus.fail("not a rex operator");
+          // return litmus.fail("not a rex operator");
         }
       };
 

diff --git a/gradle.properties b/gradle.properties
@@ -27,7 +27,7 @@ systemProp.org.gradle.internal.publish.checksums.insecure=true
 # This is version for Calcite itself
 # Note: it should not include "-SNAPSHOT" as it is automatically added by build.gradle.kts
 # Release version can be generated by using -Prelease or -Prc=<int> arguments
-calcite.version=1.30.0-kylin-4.x-r03
+calcite.version=1.30.0-kylin-4.x-r04
 # This is a version to be used from Maven repository. It can be overridden by localAvatica below
 calcite.avatica.version=1.20.0