viirya edited a comment on pull request #33913: URL: https://github.com/apache/spark/pull/33913#issuecomment-913417449
Thanks @pan3793. I tried to add lz4 wrapper classes in #33912. Fortunately only a few lz4 APIs were used at Hadoop Lz4 codec internally. So the wrapper classes are simple. It can pass the tests locally. Let me know what you think about this idea. @cloud-fan @sunchao @dongjoon-hyun Basically it sounds good as we don't need to revert shaded hadoop client related stuffs. -- 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]
