AngersZhuuuu commented on pull request #34689:
URL: https://github.com/apache/spark/pull/34689#issuecomment-978867355


   > how about the file name of the final release binaries?
   
   In `make-distribution.sh` , package name is passed by option `--name`
   ```
   if [ "$MAKE_TGZ" == "true" ]; then
     TARDIR_NAME=spark-$VERSION-bin-$NAME
     TARDIR="$SPARK_HOME/$TARDIR_NAME"
     rm -rf "$TARDIR"
     cp -r "$DISTDIR" "$TARDIR"
     tar czf "spark-$VERSION-bin-$NAME.tgz" -C "$SPARK_HOME" "$TARDIR_NAME"
     rm -rf "$TARDIR"
   fi
   ```
   
   
   In `release-build.sh`, it use parameter in `BINARY_PKGS_ARGS`
   ```
   
     if [[ $PUBLISH_SCALA_2_12 = 1 ]]; then
       echo "Packages to build: ${!BINARY_PKGS_ARGS[@]}"
       for key in ${!BINARY_PKGS_ARGS[@]}; do
         args=${BINARY_PKGS_ARGS[$key]}
         extra=${BINARY_PKGS_EXTRA[$key]}
         if ! make_binary_release "$key" "$SCALA_2_12_PROFILES $args" "$extra" 
"2.12"; then
           error "Failed to build $key package. Check logs for details."
         fi
       done
     fi
   ```
   
   In this PR I have change the `BINARY_PKGS_ARGS `. So the release package 
name should be `hadoop3.3`
   
   I have no committer authority, so can't do an end to end test. 


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to