Github user erikdw commented on a diff in the pull request: https://github.com/apache/storm/pull/736#discussion_r40755399 --- Diff: DEVELOPER.md --- @@ -222,6 +222,11 @@ To pull in a merge request you should generally follow the command line instruct # Build the code and run the tests +Storm has some oddities with packaging and plugin dependencies. We build our own plugins and shade storm-core, +but have sub-modules that depend on storm-core. Because of this the build needs to be split into two parts. --- End diff -- Can you please explain in a bit more detail why this split is needed, and how the change accomplishes that?
--- 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 infrastruct...@apache.org or file a JIRA ticket with INFRA. ---