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]