anser inline RegardsLongda------------------------------------------------------------------From:P. Taylor Goetz <[email protected]>Send Time:2015年11月5日(星期四) 04:16To:dev <[email protected]>Subject:JStorm Code Import Longda and JStorm team: I’m working on importing the JStorm code into the Apache Storm repository and have a few comments/questions. 1) Currently the top-level pom.xml builds only the `jstorm-core` and `jstorm-ui` submodules. Can the other maven submodules be removed?［longda］ok, for the other module, just remove them right now. But I think the example is very useful, It provide a lot of scenario. later we will merge it into storm-starter. 2) The `jstorm-on-yarn` module appears to be a direct copy of Yahoo’s `storm-yarn` project [1] with the packages renamed and the license headers removed. I will need to remove this.[longda] just remove it. 3) The `jstorm-utility/toplogy-monitor` appears to be a copy of a repository belonging to dianping.com [2]. I will need to remove this as it is not covered by the software grant.[longda] just remove it right now, but this is a very useful monitor topology solution. the owner of the topology-monitor is a friend of mine, later if we need it, I will try to persuade dianping to donate it. 4) The `jstorm-utility/rocket-mq` module is a git submodule pointing to https://github.com/rocketmq/rocketmq-storm. I would like to remove it.[longda] just remove it. 5) Overall, the `jstorm-utility` looks like it contains components that are add-ons not required for core functionality (and presumably why they are not part of the main build). I would like to remove that directory in the interest of focusing on merging the core functionality of the two codebases. Any useful integration/utility components can always be contributed at a later time.［longda］ i agree with you, we can later contribute them if we need them.
Let me know if you have any opinions. -Taylor [1] https://github.com/yahoo/storm-yarn[2] https://github.com/dianping/storm-util
