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.
---

Reply via email to