frankfliu commented on a change in pull request #13834: Reduce maven verbosity
which is filling up the logs of the builds
URL: https://github.com/apache/incubator-mxnet/pull/13834#discussion_r247186084
##########
File path: Makefile
##########
@@ -614,13 +614,13 @@ scalaclean:
(cd $(ROOTDIR)/scala-package && mvn clean)
scalapkg:
- (cd $(ROOTDIR)/scala-package && mvn install -DskipTests)
+ (cd $(ROOTDIR)/scala-package && mvn install -q -DskipTests)
Review comment:
I assume, developer has tested locally with maven already. If there is a
failure in CI, in most cases, error message is not enough to troubleshoot a PR
failure.
Switching -q flag to debug on CI isn't a right approach, and it's very time
consuming effort.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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