[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44874202 @d2r can you please check the latest update. I've tested on a small cluster here is a screenshot http://grab.by/xoP4 --- If your project is set up for it,

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44867544 yeah error is in returning the json here https://github.com/harshach/incubator-storm/blob/ui-rest-new/storm-core/src/clj/backtype/storm/ui/core.clj#L514 . updat

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44862608 I know what it is. The errors are doubly-nested in lists. So you have `response["errors"][0][0]["time"]` to get the time of the first error, instead of `response["

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44861584 @d2r thanks for the info. I am testing few things I'll update the PR with the fix. --- If your project is set up for it, you can reply to this email and have y

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44861386 > It looks like 9 rows with two blank cells each, and one row without any cells. Correction: It is 10 rows of two blank cells each. --- If your project is

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44860047 This is the rendered HTML I see: ```

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread harshach
Github user harshach commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44849862 @d2r component errors page is working fine. I guess its my template naming thats not good. There is div "error" which get update if the API responded with any e

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on the pull request: https://github.com/apache/incubator-storm/pull/101#issuecomment-44845150 It will be great to have an API with JSON responses coming back. I would like to resolve at least the issue with the errors table on the component page, and then I

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/101#discussion_r13285186 --- Diff: storm-core/src/clj/backtype/storm/ui/core.clj --- @@ -509,26 +231,23 @@ reverse first)]

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/101#discussion_r13285182 --- Diff: storm-core/src/clj/backtype/storm/ui/core.clj --- @@ -414,88 +218,6 @@ (defn bolt-summary? [topology s] (= :bolt (executor-summary-t

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/101#discussion_r13284947 --- Diff: storm-core/src/clj/backtype/storm/ui/core.clj --- @@ -509,26 +231,23 @@ reverse first)]

[GitHub] incubator-storm pull request: STORM-205. Add REST API to Storm UI.

2014-06-02 Thread d2r
Github user d2r commented on a diff in the pull request: https://github.com/apache/incubator-storm/pull/101#discussion_r13283946 --- Diff: storm-core/src/ui/public/component.html --- @@ -0,0 +1,88 @@ +http://www.w3.org/TR/html4/strict.dtd";> + +Storm UI + + +