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

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/07/08 15:00:07 UTC

[GitHub] [arrow] davisusanibar commented on a diff in pull request #13544: ARROW-16996: [Java] Configure Netty/GRPC/Protobuf base on BOM configuration + upgrade of dependencies by CVE

davisusanibar commented on code in PR #13544:
URL: https://github.com/apache/arrow/pull/13544#discussion_r916901429


##########
java/flight/flight-core/pom.xml:
##########
@@ -323,7 +314,6 @@
         <dependency>
           <groupId>io.netty</groupId>
           <artifactId>netty-transport-native-unix-common</artifactId>
-          <version>${dep.netty.version}</version>

Review Comment:
   What will be the impact if we only use `io.netty:netty-transport-native-unix-common:4.1.78.Final` instead of `io.netty:netty-transport-native-unix-common:osx-x86_64:4.1.78.Final`? Is there some reason to why we use that? 
   
   I am building and testing code without -osx / -linux and it finished without problems.



-- 
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: github-unsubscribe@arrow.apache.org

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