kezhuw commented on code in PR #2294:
URL: https://github.com/apache/zookeeper/pull/2294#discussion_r2278386464
##########
zookeeper-contrib/zookeeper-contrib-zooinspector/pom.xml:
##########
@@ -91,13 +91,9 @@
</dependency>
<dependency>
<groupId>ch.qos.logback</groupId>
- <artifactId>logback-core</artifactId>
- <exclusions>
- <exclusion>
- <groupId>*</groupId>
- <artifactId>*</artifactId>
- </exclusion>
- </exclusions>
+ <artifactId>logback-classic</artifactId>
+ <scope>runtime</scope>
Review Comment:
This is marked as optional runtime in ZOOKEEPER-4820(#2155). Seems to fix
test failure, but I have no cue to this. See
9b535738d80eb21fd42ae59f33b88ef291c8ec2e.
I plan to check test result for this.
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
