Skip to content

Commit

Permalink
Do not display maven version
Browse files Browse the repository at this point in the history
Remove obsolete hotfix

If maven is not downloaded for maven-wrapper,
`./mvnw` downloads it with `org.apache.maven.wrapper.Installer`.
`org.apache.maven.wrapper.Installer` in version `0.1.2-SNAPSHOT` which
we had some day before (upgraded in
3e35fc4) logs to the standard output.
The newest version of maven-wrapper.jar is `3.1.0` and it logs to log
https://mvnrepository.com/artifact/org.apache.maven.wrapper/maven-wrapper

To overcome this issue `./mvnw -v` construction was used to
skip output before any actual `./mvnw` invocation which returned
valuable text output.
  • Loading branch information
ssheikin committed Mar 14, 2022
1 parent 1399ad6 commit 2322e2c
Showing 1 changed file with 0 additions and 3 deletions.
3 changes: 0 additions & 3 deletions plugin/trino-hive-hadoop2/conf/hive-tests-defaults.sh
Original file line number Diff line number Diff line change
@@ -1,8 +1,5 @@
#!/usr/bin/env bash

# Download maven wrapper if it is not present so next command can have clean output
./mvnw --version >/dev/null

DEFAULT_DOCKER_VERSION=$(./mvnw help:evaluate -Dexpression=dep.docker.images.version -q -DforceStdout)

if [ -z "$DEFAULT_DOCKER_VERSION" ];
Expand Down

0 comments on commit 2322e2c

Please sign in to comment.