GitHub user jnioche opened a pull request:

    https://github.com/apache/storm/pull/1221

    Fix logging for LoggingMetricsConsumer STORM-584

    Currently the metrics.log file is always empty and it declared in the 
cluster.xml file whereas it should be in the worker one. The metrics end up in 
the log file which is not a huge issue but it would be cleaner to have them in 
a separate file, as originally intended. 
    
    This patch addresses this and makes so that there is one metrics file per 
worker, as per the other log files, which should address the issues mentioned 
in [https://issues.apache.org/jira/browse/STORM-584].

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/DigitalPebble/storm metricsFile

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/storm/pull/1221.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 #1221
    
----
commit 4d75ec8494eeda24d6510641259d4c469cff3ee2
Author: Julien Nioche <[email protected]>
Date:   2016-03-16T11:14:24Z

    Fix logging for LoggingMetricsConsumer STORM-584

----


---
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.
---

Reply via email to