You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@accumulo.apache.org by "cshannon (via GitHub)" <gi...@apache.org> on 2023/03/03 15:42:31 UTC

[GitHub] [accumulo] cshannon commented on pull request #3223: Update micrometer

cshannon commented on PR #3223:
URL: https://github.com/apache/accumulo/pull/3223#issuecomment-1453721986

   We could try defining our own version of Netty and import the BOM instead of relying on transitive dependencies. Also, there appears to be an exclusion for Netty for Zookeeper in core but nowhere else which is weird, especially because as far as I know we need Netty for Zookeeper to work.
   
   https://github.com/apache/accumulo/blob/fe31930ccef448a549cea082943c510b678dbecd/core/pom.xml#L127-L135
   
   Furthermore, the exclusion doesn't do anything as the GAV is incorrect as Zookeeper has a dependency on netty-handler and not netty.


-- 
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: notifications-unsubscribe@accumulo.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org