Alex Behm has posted comments on this change. ( http://gerrit.cloudera.org:8080/8150 )
Change subject: IMPALA-5990: Part 1: JNI-based LZ4 de/compression ...................................................................... Patch Set 1: I had looked at that library but concluded we would not want to pull it in as-is because it includes a lot of stuff we don't need and also the LZ4 code - but we want to use the LZ4 code from our toolchain. Reimplementing the JNI wrappers we would need seems so simple that it's not worth the trouble pulling the library and integrating it into our build/toolchain. However, looking at the library code more carefully, I think we make the implementation I posted here much simpler, so let me try and rework that. -- To view, visit http://gerrit.cloudera.org:8080/8150 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: Impala-ASF Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: I237802944875b07080db0159ff8ec548150fd95e Gerrit-Change-Number: 8150 Gerrit-PatchSet: 1 Gerrit-Owner: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Alex Behm <alex.b...@cloudera.com> Gerrit-Reviewer: Philip Zeyliger <phi...@cloudera.com> Gerrit-Comment-Date: Thu, 28 Sep 2017 05:36:14 +0000 Gerrit-HasComments: No