Skip to content

Commit

Permalink
apacheGH-42228: [CI][Java] Suppress transfer progress log in java-jars (
Browse files Browse the repository at this point in the history
apache#42230)

### Rationale for this change

Transfer progress log is meaningless and noisy in CI log.

### What changes are included in this PR?

Use the `--no-transfer-progress` option that is available Maven 3.6.1: https://maven.apache.org/ref/3.6.1/maven-embedder/cli.html

Other approach is using `-Dorg.slf4j.simpleLogger.log.org.apache.maven.cli.transfer.Slf4jMavenTransferListener=warn` like other our scripts do.

I chose `--no-transfer-progress` because it's shorter.

### Are these changes tested?

Yes.

### Are there any user-facing changes?

No.
* GitHub Issue: apache#42228

Authored-by: Sutou Kouhei <[email protected]>
Signed-off-by: Sutou Kouhei <[email protected]>
  • Loading branch information
kou authored Jun 21, 2024
1 parent 77e572b commit c9a735a
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 2 additions & 1 deletion ci/scripts/java_full_build.sh
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,8 @@ mvn clean \
-Parrow-c-data \
-Parrow-jni \
-Darrow.cpp.build.dir=$dist_dir \
-Darrow.c.jni.dist.dir=$dist_dir
-Darrow.c.jni.dist.dir=$dist_dir \
--no-transfer-progress

# copy all jar, zip and pom files to the distribution folder
find ~/.m2/repository/org/apache/arrow \
Expand Down
3 changes: 3 additions & 0 deletions dev/tasks/java-jars/github.yml
Original file line number Diff line number Diff line change
Expand Up @@ -242,6 +242,9 @@ jobs:
test -f arrow/java-dist/arrow_dataset_jni/x86_64/arrow_dataset_jni.dll
test -f arrow/java-dist/arrow_orc_jni/x86_64/arrow_orc_jni.dll
- name: Build bundled jar
env:
MAVEN_ARGS: >-
--no-transfer-progress
run: |
set -e
pushd arrow/java
Expand Down

0 comments on commit c9a735a

Please sign in to comment.