[Impala-ASF-CR] IMPALA-14061: Calcite Planner: added Calcite rules

[Anonymous Coward (Code Review)]

Anonymous Coward (816) has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/22870 )

Change subject: IMPALA-14061: Calcite Planner: added Calcite rules
......................................................................


Patch Set 4:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/22870/4/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteOptimizer.java
File 
java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteOptimizer.java:

http://gerrit.cloudera.org:8080/#/c/22870/4/java/calcite-planner/src/main/java/org/apache/impala/calcite/service/CalciteOptimizer.java@102
PS4, Line 102: RelNode preJoinOptimizedPlan = 
runOptimizeNodesProgram(relBuilder, coercedNodesPlan);
> https://github.com/feldera/feldera/blob/main/sql-to-dbsp-compiler/build.sh
in buildall.sh
# Build the SQL compiler if requested.
build_sql_compiler() {
  if [[ "$BUILD_SQL_COMPILER" -eq 1 ]]; then
    echo ">>> Building SQL compiler with Calcite version: $CALCITE_VERSION"

    # Default is to use the next version
    NEXT='y'
    CALCITE_REPO="https://github.com/apache/calcite.git";
    CALCITE_BRANCH="main"
    CALCITE_NEXT_COMMIT=""
    CALCITE_NEXT="1.40.0"
    CALCITE_CURRENT="1.39.0"

    if [[ "$CALCITE_VERSION" = "current" ]]; then
      NEXT='n'
    fi

    update_pom() {
      VERSION=$1
      sed -i -E 
"s/<calcite.version>(.*)<\/calcite.version>/<calcite.version>${VERSION}<\/calcite.version>/"
 ./java/calcite-planner/pom.xml
    }

    if [ "${NEXT}" = 'y' ]; then
      update_pom "${CALCITE_NEXT}"
      pushd /tmp >/dev/null
      if [ -d "calcite" ]; then
        echo "Removing existing calcite directory..."
        rm -rf calcite
      fi

      if [[ ! -z "${CALCITE_NEXT_COMMIT}" ]]; then
        GIT_ARGS="--no-checkout"
      else
        GIT_ARGS="--depth 1"
      fi
      git clone --quiet --single-branch --branch "${CALCITE_BRANCH}" 
${GIT_ARGS} "${CALCITE_REPO}"
      cd calcite
      if [[ ! -z "${CALCITE_NEXT_COMMIT}" ]]; then
        git fetch origin "${CALCITE_NEXT_COMMIT}"
        git checkout "${CALCITE_NEXT_COMMIT}"
      fi

      GROUP=org.apache.calcite
      VERSION="${CALCITE_NEXT}"
      ./gradlew build -x test -x checkStyleMain -x autoStyleJavaCheck build 
--console=plain -Dorg.gradle.logging.level=quiet

      for DIR in core babel server linq4j
      do
          ARTIFACT=calcite-${DIR}
          mvn install:install-file 
-Dfile=${DIR}/build/libs/${ARTIFACT}-${VERSION}-SNAPSHOT.jar -DgroupId=${GROUP} 
-DartifactId=${ARTIFACT} -Dversion=${VERSION} -Dpackaging=jar 
-DgeneratePom=true -q -B
      done
      popd >/dev/null
      rm -rf /tmp/calcite
    else
      update_pom "${CALCITE_CURRENT}"
    fi
  fi
}



--
To view, visit http://gerrit.cloudera.org:8080/22870
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: Impala-ASF
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6671f7ed298a18965ef0b7a5fc10f4912333a52b
Gerrit-Change-Number: 22870
Gerrit-PatchSet: 4
Gerrit-Owner: Steve Carlin <scar...@cloudera.com>
Gerrit-Reviewer: Aman Sinha <amsi...@cloudera.com>
Gerrit-Reviewer: Anonymous Coward (816)
Gerrit-Reviewer: Fang-Yu Rao <fangyu....@cloudera.com>
Gerrit-Reviewer: Impala Public Jenkins <impala-public-jenk...@cloudera.com>
Gerrit-Reviewer: Joe McDonnell <joemcdonn...@cloudera.com>
Gerrit-Reviewer: Michael Smith <michael.sm...@cloudera.com>
Gerrit-Reviewer: Riza Suminto <riza.sumi...@cloudera.com>
Gerrit-Reviewer: Steve Carlin <scar...@cloudera.com>
Gerrit-Comment-Date: Fri, 16 May 2025 10:18:10 +0000
Gerrit-HasComments: Yes