Github user revans2 commented on the pull request:
https://github.com/apache/storm/pull/600#issuecomment-115845268
@harshach which exclude are you talking about? If you want to make a change
quickly I am OK with that. I am just not totally sure what the impact that
this is going to have. Will we now have commons-codec as a dependency that we
didn't before? Will we now have a version of hadoop-auth with classes
rewritten to point to the shaded commons-codec?
If it is the former, then we will be stuck with commons-codec being a
dependency until we go to another major version. If it is the former I am a
bit nervous about how hadoop might react to it, but not very nervous.
I am fine either way, I just want to know what the impact is, and then have
someone put together a pull request we can review it and check it in.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at [email protected] or file a JIRA ticket
with INFRA.
---
