viirya commented on a change in pull request #3385:
URL: https://github.com/apache/hadoop/pull/3385#discussion_r706425782
##########
File path: hadoop-client-modules/hadoop-client-runtime/pom.xml
##########
@@ -155,6 +155,8 @@
<exclude>org.ow2.asm:*</exclude>
<!-- Leave bouncycastle unshaded because it's signed
with a special Oracle certificate so it can be a custom JCE security provider
-->
<exclude>org.bouncycastle:*</exclude>
+ <!-- Leave snappy that includes native methods which
cannot be relocated. -->
+ <exclude>org.xerial.snappy:*</exclude>
Review comment:
Okay. So not like `optional` which seems can be inherited, we need to
make it runtime explicitly.
--
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]
