pkuwm commented on a change in pull request #1377:
URL: https://github.com/apache/helix/pull/1377#discussion_r491208210
##########
File path: helix-rest/pom.xml
##########
@@ -148,6 +148,12 @@
<type>test-jar</type>
<scope>test</scope>
</dependency>
+ <dependency>
+ <groupId>io.dropwizard.metrics</groupId>
+ <artifactId>metrics-jersey2</artifactId>
Review comment:
As long as users are not using ivy, it is fine because we've defined all
the required dependencies in pom. If they are using ivy without adding
dependencies, it's been a problem because dependencies like `jersey-server` are
not even in rest ivy.
----------------------------------------------------------------
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]
