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

Posted to dev@kafka.apache.org by KevinLiLu <gi...@git.apache.org> on 2017/09/15 05:52:52 UTC

[GitHub] kafka pull request #3867: MINOR: Change metric.reporters description to use ...

GitHub user KevinLiLu opened a pull request:

    https://github.com/apache/kafka/pull/3867

    MINOR: Change metric.reporters description to use actual MetricsReporter.class name

    The `metric.reporters` description in the documentation says to implement the `MetricReporter`, but the actual class is `MetricsReporter`. 
    
    The broker configuration section also uses the client description so it says to implement the `MetricReporter` interface, but the actual class is `KafkaMetricsReporter`.

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

    $ git pull https://github.com/KevinLiLu/kafka trunk

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

    https://github.com/apache/kafka/pull/3867.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 #3867
    
----
commit 9fd55293ae12463d593b7d7ab769ec22c222002d
Author: Kevin Lu <ke...@paypal.com>
Date:   2017-09-15T05:35:36Z

    MINOR: Change metric.reporters description to use actual MetricsReporter.class name

----


---

[GitHub] kafka pull request #3867: MINOR: Use full package name when classes referenc...

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

Github user KevinLiLu closed the pull request at:

    https://github.com/apache/kafka/pull/3867


---