viirya commented on code in PR #15267:
URL: https://github.com/apache/arrow/pull/15267#discussion_r1064959196


##########
ci/scripts/java_full_build.sh:
##########
@@ -61,7 +61,13 @@ mvn clean \
 
 # copy all jar, zip and pom files to the distribution folder
 find . \
-     "(" -name "*-javadoc.jar" -o -name "*-sources.jar" ")" \
+     "(" \
+     -name "*.jar" -o \
+     -name "*.json" -o \
+     -name "*.pom" -o \
+     -name "*.xml" -o \
+     -name "*.zip" \
+     ")" \
      -exec echo {} ";" \
      -exec cp {} $dist_dir ";"
 find ~/.m2/repository/org/apache/arrow \

Review Comment:
   Hmm, I think the previous comment diff points to L73 instead of L63?
   
   ```diff
   diff --git a/ci/scripts/java_full_build.sh b/ci/scripts/java_full_build.sh
   index 1c07971bcc..2734f3e9db 100755
   --- a/ci/scripts/java_full_build.sh
   +++ b/ci/scripts/java_full_build.sh
   @@ -65,7 +65,13 @@ find . \
         -exec echo {} ";" \
         -exec cp {} $dist_dir ";"
    find ~/.m2/repository/org/apache/arrow \
   -     "(" -name "*.jar" -o -name "*.zip" -o -name "*.pom" ")" \
   +     "(" \
   +     -name "*.jar" -o \
   +     -name "*.json" -o \
   +     -name "*.pom" -o \
   +     -name "*.xml" -o \
   +     -name "*.zip" \
   +     ")" \
         -exec echo {} ";" \
         -exec cp {} $dist_dir ";"
   ```
   
   I.e., the one finding the files in local repo dir.



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