flink-issues mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF GitHub Bot (JIRA)" <j...@apache.org>
Subject [jira] [Commented] (FLINK-4564) [metrics] Delimiter should be configured per reporter
Date Mon, 26 Sep 2016 14:06:20 GMT

    [ https://issues.apache.org/jira/browse/FLINK-4564?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15523163#comment-15523163

ASF GitHub Bot commented on FLINK-4564:

Github user zentol commented on a diff in the pull request:

    --- Diff: docs/monitoring/metrics.md ---
    @@ -280,6 +280,7 @@ Metrics can be exposed to an external system by configuring one or
several repor
     - `metrics.reporter.<name>.<config>`: Generic setting `<config>` for
the reporter named `<name>`.
     - `metrics.reporter.<name>.class`: The reporter class to use for the reporter named
     - `metrics.reporter.<name>.interval`: The reporter interval to use for the reporter
named `<name>`.
    +- `metrics.reporter.<name>.scope.delimiter`: The delimiter for to use for the identifier
(default value use `metrics.scope.delimiter`) for the reporter named `<name>`.
    --- End diff --
    for to use -> to use

> [metrics] Delimiter should be configured per reporter
> -----------------------------------------------------
>                 Key: FLINK-4564
>                 URL: https://issues.apache.org/jira/browse/FLINK-4564
>             Project: Flink
>          Issue Type: Bug
>          Components: Metrics
>    Affects Versions: 1.1.0
>            Reporter: Chesnay Schepler
>            Assignee: Anton Mushin
> Currently, the delimiter used or the scope string is based on a configuration setting
shared by all reporters. However, different reporters may have different requirements in regards
to the delimiter, as such we should allow reporters to use a different delimiter.
> We can keep the current setting as a global setting that is used if no specific setting
was set.

This message was sent by Atlassian JIRA

View raw message