commit 4093975e2d5da61c1ef6217796a1543a602b1d40
author Csaba Ringhofer <csringhofer@cloudera.com> Thu Dec 05 20:18:56 2019 +0100
committer Csaba Ringhofer <csringhofer@cloudera.com> Mon Dec 09 11:55:23 2019 +0000
tree c3fabdaa47c2d38b6c4cf376c35c096a5844ffb2
parent f33a9d0d426f2cbaaf225d7ea08b15966e537f31

Update mvn snapshot dependencies in "clean" checkouts

buildall.sh saves the cdh/cdp version into .cdh/.cdp, and updates
the dependencies if this doesn't match the version from config.

This lead to updating the dependencies when switching to a different
checkout in the same directory, but didn't do this in a fresh checkout,
which could lead to build issues when the .m2 cache was dirty.

Note that this doesn't protect from switching between Impala directories
with different cdh/cdp versions.

Change-Id: I8bbde17e7c97466391aa20ac3d59c6943e7f7256
Reviewed-on: http://gerrit.cloudera.org:8080/14854
Reviewed-by: Zoltan Borok-Nagy <boroknagyz@cloudera.com>
Tested-by: Impala Public Jenkins <impala-public-jenkins@cloudera.com>
Reviewed-by: Laszlo Gaal <laszlo.gaal@cloudera.com>

buildall.sh

diff --git a/buildall.sh b/buildall.sh
index 23ea8f9..ab7d415 100755
--- a/buildall.sh
+++ b/buildall.sh
@@ -547,11 +547,10 @@
 # local cache.
 CDH_FILE="${IMPALA_HOME}/.cdh"
 CDP_FILE="${IMPALA_HOME}/.cdp"
-if [[ -f ${CDH_FILE} && -f ${CDP_FILE} ]]; then
-  if [[ $(cat ${CDH_FILE}) != ${CDH_BUILD_NUMBER} || \
-        $(cat ${CDP_FILE}) != ${CDP_BUILD_NUMBER} ]]; then
-    export IMPALA_MAVEN_OPTIONS="${IMPALA_MAVEN_OPTIONS} -U"
-  fi
+if [[ ! -f ${CDH_FILE} || ! -f ${CDP_FILE} || \
+      $(cat ${CDH_FILE}) != ${CDH_BUILD_NUMBER} || \
+      $(cat ${CDP_FILE}) != ${CDP_BUILD_NUMBER} ]]; then
+  export IMPALA_MAVEN_OPTIONS="${IMPALA_MAVEN_OPTIONS} -U"
 fi
 echo "${CDH_BUILD_NUMBER}" > ${CDH_FILE}
 echo "${CDP_BUILD_NUMBER}" > ${CDP_FILE}