You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@samza.apache.org by rmatharu <gi...@git.apache.org> on 2018/08/17 23:39:47 UTC

[GitHub] samza pull request #610: SAMZA-1733 : Adding comments, adding emptyness chec...

GitHub user rmatharu opened a pull request:

    https://github.com/apache/samza/pull/610

    SAMZA-1733 : Adding comments, adding emptyness check to MetricsSnapshotReporter message

    * Added comments for MetricsConfig 
    * Added simple emptyness check for MetricsSnapshotReporter

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

    $ git pull https://github.com/rmatharu/samza bugfix-emptymessage

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

    https://github.com/apache/samza/pull/610.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 #610
    
----
commit 3481ce22aa7be443aad09b55ca01ee9f226e13f9
Author: rmatharu@linkedin.com <rm...@...>
Date:   2018-08-17T23:37:23Z

    Adding comments, preveting MetricsSnapshotReporter from publishing empty messages in case of blacklist

----


---

[GitHub] samza pull request #610: SAMZA-1733 : Adding comments, adding emptyness chec...

Posted by asfgit <gi...@git.apache.org>.

Github user asfgit closed the pull request at:

    https://github.com/apache/samza/pull/610


---