adoroszlai commented on a change in pull request #2945:
URL: https://github.com/apache/ozone/pull/2945#discussion_r811620164



##########
File path: hadoop-ozone/common/pom.xml
##########
@@ -48,6 +48,17 @@ https://maven.apache.org/xsd/maven-4.0.0.xsd";>
     <dependency>
       <groupId>io.netty</groupId>
       <artifactId>netty-handler-proxy</artifactId>
+    </dependency>
+      <dependency>
+        <groupId>io.netty</groupId>
+        <artifactId>netty-tcnative-boringssl-static</artifactId>
+        <version>2.0.38.Final</version> <!-- See table for correct version -->
+        <scope>runtime</scope>
+      </dependency>
+    <dependency>
+      <groupId>io.netty</groupId>
+      <artifactId>netty-tcnative</artifactId>
+      <version>2.0.38.Final</version> <!-- See table for correct version -->

Review comment:
       I think it's this one: 
https://github.com/grpc/grpc-java/blob/master/SECURITY.md#netty
   
   There is a compatibility table, describing which netty and tcnative versions 
to use for each gRPC version.  Netty and tcnative usually need to be bumped 
together.




-- 
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]



---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to