dbtsai commented on a change in pull request #2159:
URL: https://github.com/apache/hadoop/pull/2159#discussion_r458446690



##########
File path: hadoop-project/pom.xml
##########
@@ -1727,6 +1728,11 @@
         <artifactId>jna</artifactId>
         <version>${jna.version}</version>
       </dependency>
+      <dependency>

Review comment:
       @steveloughran we would like to actually bundle this jar into common 
since this is a very clean jar and used in many projects such as presto and orc 
to provide their compression codecs. 
   
   In fact, we would like to have snappy to fallback to aircompressor if no 
native lib is provided. For many of our devs, it's non-trivial to setup native 
libs in their development env since it requires to compile many native libs 
from sources and install them into LD path. If we can fallback to pure java 
snappy implementation in aircompressor, it will make the developers' life a way 
easier. 




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

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