elek commented on a change in pull request #479: HDDS-2925. Remove hdfs-client
dependency from hdds-common
URL: https://github.com/apache/hadoop-ozone/pull/479#discussion_r370642521
##########
File path: hadoop-ozone/dev-support/checks/unit.sh
##########
@@ -17,7 +17,7 @@ DIR="$( cd "$( dirname "${BASH_SOURCE[0]}" )" >/dev/null
2>&1 && pwd )"
cd "$DIR/../../.." || exit 1
export MAVEN_OPTS="-Xmx4096m"
-mvn -B -DskipShade -Dskip.yarn -fae test -pl \!:hadoop-ozone-integration-test
"$@"
+mvn -B -DskipShade -Dskip.yarn -fae install -pl
\!:hadoop-ozone-integration-test "$@"
Review comment:
I tried to understand why the unit tests are failed and I was afraid that
some changes are missing because the missing install phase. But I think it was
because the tricky provided dependency between hdfs server and hdfs.
I reverted the change.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
