Impala Public Jenkins has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/24473 )
Change subject: IMPALA-15105: Don't package test configuration files in calcite-planner ...................................................................... IMPALA-15105: Don't package test configuration files in calcite-planner The calcite-planner module declares fe/src/test/resources as a maven resource directory, which causes all configuration files (e.g., hive-site.xml) for testing are packaged into the jar file. They could be used in production, which overwrites the real configuration files. This patch removes it and modifies TpcdsCpuCostPlannerTest to copy the configuration files in classpath directories, so don't need them in the jar file. Testing - Ran Java unitests in calcite-planner. Assisted-by: Opus 4.8 (Cursor) Change-Id: I2780ad8fc9d0c8b9f1a1962dc0f9eb7d167b1783 Reviewed-on: http://gerrit.cloudera.org:8080/24473 Reviewed-by: Impala Public Jenkins <[email protected]> Tested-by: Impala Public Jenkins <[email protected]> --- M java/calcite-planner/pom.xml M java/calcite-planner/src/test/java/org/apache/impala/calcite/planner/TpcdsCpuCostPlannerTest.java 2 files changed, 18 insertions(+), 18 deletions(-) Approvals: Impala Public Jenkins: Looks good to me, approved; Verified -- To view, visit http://gerrit.cloudera.org:8080/24473 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I2780ad8fc9d0c8b9f1a1962dc0f9eb7d167b1783 Gerrit-Change-Number: 24473 Gerrit-PatchSet: 3 Gerrit-Owner: Quanlong Huang <[email protected]> Gerrit-Reviewer: Impala Public Jenkins <[email protected]> Gerrit-Reviewer: Peter Rozsa <[email protected]> Gerrit-Reviewer: Quanlong Huang <[email protected]> Gerrit-Reviewer: Steve Carlin <[email protected]>
