[
https://issues.apache.org/jira/browse/STORM-1524?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15131255#comment-15131255
]
ASF GitHub Bot commented on STORM-1524:
---------------------------------------
GitHub user kishorvpatil opened a pull request:
https://github.com/apache/storm/pull/1075
[STORM-1524] Add Pluggable daemon statistics - Metrics Reporters
- This patch adds configuration -
`storm.statistics.preparable.reporter.plugin` a list of plugins that implement
`PreparableReporter` interface.
- This defaults to `JmxPreparableReporter` ( i.e. current behavior)
- Implementation for `CsvPreparableReporter`, `JmxPreparableReporter` and
`ConsolePreparableReporter`.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kishorvpatil/incubator-storm storm1524
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/storm/pull/1075.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #1075
----
commit 46ef55890cebf22807e0dc293fb665ba85caeabf
Author: Kishor Patil <kpa...@yahoo-inc.com>
Date: 2016-02-03T16:49:56Z
Create stats plugin for JMX
Add config for stats reporter plugin and use it
Use Regular Map instead of Config in interface
Adding log entries for statiscs plugin actions.
commit d86b99b56902d5c9eae1aaeed369bf3f7e611853
Author: Kishor Patil <kpa...@yahoo-inc.com>
Date: 2016-02-03T19:47:09Z
Adding Cvs and Console statistics reporter plugins
Make statistics reporter plugins a list.
----
> Make Storm daemon function statistics reporter pluggable
> --------------------------------------------------------
>
> Key: STORM-1524
> URL: https://issues.apache.org/jira/browse/STORM-1524
> Project: Apache Storm
> Issue Type: New Feature
> Components: storm-core
> Affects Versions: 0.10.0, 1.0.0, 0.10.1
> Reporter: Kishor Patil
> Assignee: Kishor Patil
>
> We use codahale, metrics-clojure to gather daemon side stats, but currently
> the we have only three reporters available which use builder pattern. So it
> would be useful to have ability to plugin different reporters that can use
> configuration instead of builder pattern.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
