Merge pull request #341 from gleonSun/kycalcite-1.30.0.x-Debug

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

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

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