vibhatha commented on code in PR #40514:
URL: https://github.com/apache/arrow/pull/40514#discussion_r1531945825
##########
dev/release/verify-release-candidate.sh:
##########
@@ -76,6 +76,7 @@ esac
# which is different from the ARROW_SOURCE_DIR set in ensure_source_directory()
SOURCE_DIR="$(cd "$(dirname "${BASH_SOURCE[0]:-$0}")" && pwd)"
ARROW_DIR="$(cd "${SOURCE_DIR}/../.." && pwd)"
+MAVEN_VERSION=3.8.7
Review Comment:
I had to hard-code this to the script. Wouldn't be very intuitive if it was
made an input as this is a generic script for verification, right? Is there a
better way? Maybe the container create an environmental variable and we use it
here?
cc @raulcd @kou
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
