saintstack commented on pull request #2297: URL: https://github.com/apache/hadoop/pull/2297#issuecomment-700960375
> happy with that? Thanks @steveloughran If I understand you correctly, because you figured we actually already include snappy-java and that snappy is already a hadoop-common dependency, then making it so snappy-java is 'compile' rather than 'provided' is ok by you. If so, yeah, I think this better. Operators don't have to make sure of native snappy everywhere since snappy-java provides it. (Looks like @viirya has gone ahead and made snappy-java compile scope...) ---------------------------------------------------------------- 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]
