Steve Carlin has posted comments on this change. ( http://gerrit.cloudera.org:8080/21109 )
Change subject: IMPALA-12872: Use Calcite for optimization - part 1: simple queries ...................................................................... Patch Set 5: (1 comment) http://gerrit.cloudera.org:8080/#/c/21109/5/fe/pom.xml File fe/pom.xml: http://gerrit.cloudera.org:8080/#/c/21109/5/fe/pom.xml@616 PS5, Line 616: <dependency> : <groupId>org.apache.calcite</groupId> : <artifactId>calcite-core</artifactId> : <version>1.36.0</version> : </dependency> : <dependency> : <groupId>org.apache.calcite.avatica</groupId> : <artifactId>avatica-core</artifactId> : <version>1.23.0</version> : </dependency> > Yeah. This was an interesting dilemma for me Ok, I think I made the changes that make this better. The dependencies are now in the java/calcite-planner/target directory and the path is set up via set-classpath through an environment variable. The CLASSPATH is set up similar to how fe sets up the CLASSPATH. The custom cluster unit test is now changed too. -- To view, visit http://gerrit.cloudera.org:8080/21109 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I453fd75b7b705f4d7de1ed73c3e24cafad0b8c98 Gerrit-Change-Number: 21109 Gerrit-PatchSet: 5 Gerrit-Owner: Steve Carlin <[email protected]> Gerrit-Reviewer: Aman Sinha <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Joe McDonnell <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]> Gerrit-Comment-Date: Thu, 21 Mar 2024 00:31:55 +0000 Gerrit-HasComments: Yes
