vibhatha commented on code in PR #40514:
URL: https://github.com/apache/arrow/pull/40514#discussion_r1533169538


##########
dev/release/verify-release-candidate.sh:
##########
@@ -468,6 +468,22 @@ install_conda() {
   conda activate base
 }
 
+maybe_setup_maven() {
+  MAVEN_VERSION=3.8.7
+  SYSTEM_MAVEN_VERSION=$(mvn -v | head -n 1 | awk '{print $3}')
+  if [[ $(echo "$SYSTEM_MAVEN_VERSION $MAVEN_VERSION" | awk '{print ($1 < 
$2)}') -eq 1 ]]; then

Review Comment:
   `3.0.0-gamma-13` merely using as an example to test a point. 
   
   > If so, we should not use the version in MAVEN_VERSION. We should use only 
released Maven.
   
   Yes, we can agree to this and only verify released versions.  



-- 
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]

Reply via email to