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 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 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 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 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 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 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 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 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 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 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 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
+
+
+
12 matches
Mail list logo