Github user tedxia commented on the pull request:
https://github.com/apache/storm/pull/600#issuecomment-115479231
@revans2 thanks for your replay. But I still think we must add storm-hdfs
and all of its dependencies to lib or extlib, and storm user must add
storm-hdfs as topology's dependency as it use HdfsBolt and so on. The reason
why we should add storm-hdfs to daemon's CLASS_PATH is that daemons such as
nimbus and supervisor will directly call AutoHDFSâs interface. I think it is
a good idea if we add storm-hdfs and storm-hbase and all of their dependencies
to extlib-daemon by default. The jars that have been shaded by storm-core must
also add to extlib-daemon, if they are depended by storm-hdfs or storm-hbase,
just because AutoHDFS and AutoHBase load by class loader and they can't
identify the shaded jars.
---
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.
---
