Apache9 commented on code in PR #5475:
URL: https://github.com/apache/hbase/pull/5475#discussion_r1372695926
##########
hbase-common/pom.xml:
##########
@@ -97,6 +97,15 @@
<groupId>io.opentelemetry</groupId>
<artifactId>opentelemetry-semconv</artifactId>
</dependency>
+ <!-- include netty dependencies as hadoop needs them-->
Review Comment:
Ah, shit, we only depends on hadoop-common in hbase-common, not hadoop-hdfs.
Hadoop-common does not depend on netty directly but it depends on zookeeper, as
while as curator...
Let me think how to only set necessary dependencies here...
--
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]
