update condition

apache · Dec 13, 2024 · 55d17bd · 55d17bd
1 parent 5f737f8
commit 55d17bd
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 221 deletions.
diff --git a/packaging/bundle-validation/ci_maven.sh b/packaging/bundle-validation/ci_maven.sh
diff --git a/packaging/bundle-validation/ci_run.sh b/packaging/bundle-validation/ci_run.sh
@@ -39,17 +39,17 @@ echo "MAVEN_BASE_URL: $MAVEN_BASE_URL"
 echo "STAGING_REPO_NUM: $STAGING_REPO_NUM"
 
 # Ensure only one of STAGING_REPO_NUM or MAVEN_BASE_URL is provided
-if [[ -z "$STAGING_REPO_NUM" && -z "$MAVEN_BASE_URL" ]]; then
+if [[ -n "$STAGING_REPO_NUM" && -n "$MAVEN_BASE_URL" ]]; then
   echo "Error: Both STAGING_REPO_NUM and MAVEN_BASE_URL cannot be provided simultaneously."
   exit 1
 fi
 
-if [[ -z "$STAGING_REPO_NUM" ]]; then
+if [[ -n "$STAGING_REPO_NUM" ]]; then
   REPO_BASE_URL=https://repository.apache.org/content/repositories/orgapachehudi-$STAGING_REPO_NUM/org/apache/hudi
-  echo "Downloading bundle jars from staging repo orgapachehudi-$STAGING_REPO_NUM ..."
-elif [[ -z "$MAVEN_BASE_URL" ]]; then
+  echo "Downloading bundle jars from staging repo orgapachehudi-$REPO_BASE_URL ..."
+elif [[ -n "$MAVEN_BASE_URL" ]]; then
   REPO_BASE_URL=$MAVEN_BASE_URL/org/apache/hudi
-  echo "Downloading bundle jars from maven central - $MAVEN_BASE_URL ..."
+  echo "Downloading bundle jars from maven central - $REPO_BASE_URL ..."
 else
   echo "Error: Either STAGING_REPO_NUM or MAVEN_BASE_URL must be provided."
   exit 1
@@ -132,7 +132,7 @@ fi
 TMP_JARS_DIR=/tmp/jars/$(date +%s)
 mkdir -p $TMP_JARS_DIR
 
-if [ -z "$STAGING_REPO_NUM" ] || [ -z "$MAVEN_BASE_URL" ]; then
+if [ -z "$STAGING_REPO_NUM" ] && [ -z "$MAVEN_BASE_URL" ]; then
   echo 'Adding built bundle jars for validation'
   if [[ "$SCALA_PROFILE" != 'scala-2.13' ]]; then
     # For Scala 2.13, Flink is not support, so skipping the Flink bundle validation