This is an automated email from the ASF dual-hosted git repository.
joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git
The following commit(s) were added to refs/heads/master by this push:
new 6b93f0957 IMPALA-14711: Ensure Calcite planner jar file gets built,
even if tests fail.
6b93f0957 is described below
commit 6b93f0957307826bc28e9582a3d69a43357959c5
Author: Steve Carlin <[email protected]>
AuthorDate: Thu Jan 29 16:52:28 2026 -0800
IMPALA-14711: Ensure Calcite planner jar file gets built, even if tests
fail.
If the Calcite junit tests fail, the jar file currently does not
get built. Without the existence of the jar, some e2e tests to fail,
making it harder to figure out the root cause. Therefore we should
always ensure the Calcite jar file gets built.
Change-Id: If3fe2b3d652dea55f3b14467e605def62da79da3
Reviewed-on: http://gerrit.cloudera.org:8080/23920
Tested-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Michael Smith <[email protected]>
Reviewed-by: Joe McDonnell <[email protected]>
---
bin/run-all-tests.sh | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/bin/run-all-tests.sh b/bin/run-all-tests.sh
index c1b0702a3..406257564 100755
--- a/bin/run-all-tests.sh
+++ b/bin/run-all-tests.sh
@@ -318,13 +318,13 @@ do
pushd ${IMPALA_HOME}/java/calcite-planner
# Install Calcite FE jar
- "${IMPALA_HOME}/bin/mvn-quiet.sh" -fae install -DskipTests
if [[ "${TARGET_FILESYSTEM}" != "s3" ]]; then
# Run Calcite FE tests
if ! "${IMPALA_HOME}/bin/mvn-quiet.sh" -fae test install ; then
TEST_RET_CODE=1
fi
fi
+ "${IMPALA_HOME}/bin/mvn-quiet.sh" -fae install -DskipTests
popd
# Run the FE custom cluster tests only if not running against S3
