Github user ptgoetz commented on the pull request:
https://github.com/apache/incubator-storm/pull/101#issuecomment-42307190
I think this is a pretty good start. Switching over to a REST API will
enable external systems to do monitoring, visualizations, etc.
However, there are a couple of issues that need to be addressed:
Main Screen:
* Topology Summary: uptime is not formatted Xm Xs
Topology Detail:
* Topology stats: # acked is always 0
* Spouts: Transferred and Acked fields are blank
* Bolts: Transferred, Execute Latency, and Process Latency are blank
* Capacity values are off on all screens
In `core.clj`:
* URL decoding should use the `url-decode` function from
`backtype.storm.util.clj` to enforce the use of `UTF-8` encoding.
---
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 [email protected] or file a JIRA ticket
with INFRA.
---
