[GitHub] storm pull request: [STORM-1252] port backtype.storm.stats to java

[unsleepy22 <gi...@git.apache.org>]

Github user unsleepy22 commented on the pull request:

    https://github.com/apache/storm/pull/1147#issuecomment-190526220
  
    @revans2 yes I see the exception too when running your test, but looks odd because I've tested all storm ui pages in distributed mode with java stats and everything looks good. The exception is as follows, StatsUtil.filterSysStreams expects a map of {window -> stream id -> value}, while in your test it gets a map of {stream id -> value}. I suppose this happens in local mode only? Though I can manage to make through your test by changing this method a little.


---
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 infrastructure@apache.org or file a JIRA ticket
with INFRA.
---