[GitHub] storm pull request: STORM-598:Newly submitted topologies do not sh...

2015-01-25 Thread caofangkun
Github user caofangkun closed the pull request at: https://github.com/apache/storm/pull/355 --- 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

[GitHub] storm pull request: STORM-598:Newly submitted topologies do not sh...

2015-01-25 Thread caofangkun
Github user caofangkun commented on the pull request: https://github.com/apache/storm/pull/355#issuecomment-71407839 @revans2 I have tried on master. The UI indeed display correctly. This should be no longer a issue. Thanks for your attention and patience. I will close

[GitHub] storm pull request: STORM-598:Newly submitted topologies do not sh...

2015-01-23 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/355#issuecomment-71263582 The code compiles now, but I still have not been able to reproduce the original cause of this problem. am I doing something wrong? ``` ./bin/storm dev-zookeeper

[GitHub] storm pull request: STORM-598:Newly submitted topologies do not sh...

2014-12-23 Thread revans2
Github user revans2 commented on a diff in the pull request: https://github.com/apache/storm/pull/355#discussion_r22236169 --- Diff: storm-core/src/clj/backtype/storm/daemon/nimbus.clj --- @@ -1248,7 +1256,7 @@ (when-let [owner

[GitHub] storm pull request: STORM-598:Newly submitted topologies do not sh...

2014-12-23 Thread revans2
Github user revans2 commented on the pull request: https://github.com/apache/storm/pull/355#issuecomment-68001550 Also I tried to reproduce the issue on master and I am seeing the second topology come up on the ui without any issues. Could you try to reproduce the error on master

[GitHub] storm pull request: STORM-598:Newly submitted topologies do not sh...

2014-12-22 Thread caofangkun
GitHub user caofangkun opened a pull request: https://github.com/apache/storm/pull/355 STORM-598:Newly submitted topologies do not show up on the storm ui cluser page when the Storm Cluster run out of worker slots 1: Set up a Storm Cluster with 1 Supervisor(4 worker slots) 2: