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

    https://github.com/apache/flink/pull/5099#discussion_r156052191
  
    --- Diff: docs/monitoring/metrics.md ---
    @@ -329,11 +329,11 @@ or by assigning unique names to jobs and operators.
     Metrics can be exposed to an external system by configuring one or several 
reporters in `conf/flink-conf.yaml`. These
     reporters will be instantiated on each job and task manager when they are 
started.
     
    -- `metrics.reporters`: The list of named reporters.
     - `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 `<name>`.
     - `metrics.reporter.<name>.interval`: The reporter interval to use for the 
reporter named `<name>`.
     - `metrics.reporter.<name>.scope.delimiter`: The delimiter to use for the 
identifier (default value use `metrics.scope.delimiter`) for the reporter named 
`<name>`.
    +- `metrics.reporters`: (optional) An include list for reporters to 
instantiate. By default all configured reporters will be used.
    --- End diff --
    
    I guess it can't hurt to explicitly mention "comma-separated", but I don't 
have a strong preference, we have an example after all.


---

Reply via email to