pkuwm commented on a change in pull request #1348:
URL: https://github.com/apache/helix/pull/1348#discussion_r483313347
##########
File path: build
##########
@@ -19,4 +19,4 @@
# under the License.
#
-mvn clean install -Dmaven.test.skip=true
+mvn clean install -Dmaven.test.skip.exec=true
Review comment:
I removed compiling before to improve the build time, because readme
also doesn't compile the tests. Then today I was aware that helix admin webapp
depends on helix-core test..
If you remove the local mvn cached jars and build, it would fail.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
