GitHub user HeartSaVioR opened a pull request:

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

    STORM-1700 Introduce 'whitelist' / 'blacklist' option to MetricsConsumer

    > NOTE: This is based on STORM-1698, PR #1322 
    
    * Users can set whitelist or blacklist to filter out metrics by name
      * if none of them specified (by default), no metrics are filtered out
    * how to match: substring match with regular expression
      * use both ^ and $ when you want to match strictly (full string match)
    * added unit test
    
    I originally worked based on 1.x-branch and cherry-picked to other branch 
based on master. 
    During cherry-picking, though I remove the changeset of common.clj, git 
said this file is modified.
    If we want to remove this change, I'll add commit to fix this.

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

    $ git pull https://github.com/HeartSaVioR/storm STORM-1700

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

    https://github.com/apache/storm/pull/1324.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 #1324
    
----
commit e1db0db86eeb3857bc6f958ef0af81d06ff0d9af
Author: Jungtaek Lim <[email protected]>
Date:   2016-04-08T04:05:00Z

    STORM-1698 Asynchronous MetricsConsumerBolt
    
    * change MetricsConsumerBolt's behavior to asynchronus manner
      * to avoid bad side effect of topology
    * for details please refer JIRA issue: 
https://issues.apache.org/jira/browse/STORM-1698

commit 960fe8aeb6ea22523c2516b6fc79f61f7ae0d387
Author: Jungtaek Lim <[email protected]>
Date:   2016-04-09T05:11:20Z

    STORM-1700 Introduce 'whitelist' / 'blacklist' option to MetricsConsumer
    
    * Users can set whitelist or blacklist to filter out metrics by name
      * if none of them specified (by default), no metrics are filtered out
    * how to match: substring match with regular expression
      * use both ^ and $ when you want to match strictly (full string match)
    * added unit test

----


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