ChengkaiYang2022 commented on a change in pull request #19023:
URL: https://github.com/apache/flink/pull/19023#discussion_r833348090
##########
File path: docs/content.zh/docs/deployment/metric_reporters.md
##########
@@ -153,42 +152,46 @@ metrics.reporter.influxdb.writeTimeout: 60000
metrics.reporter.influxdb.interval: 60 SECONDS
```
-The reporter would send metrics using http protocol to the InfluxDB server
with the specified retention policy (or the default policy specified on the
server).
-All Flink metrics variables (see [List of all Variables]({{< ref
"docs/ops/metrics" >}}#list-of-all-variables)) are exported as InfluxDB tags.
+InfluxDB 发送器会使用 http 协议按照将指标发送到 InfluxDB 服务器。指标的保留策略可通过配置指定，或按照 InfluxDB
服务端的保留策略决定。
+所有的 Flink 运行指标变量（见 [List of all Variables]({{< ref "zh/docs/ops/metrics"
>}}#list-of-all-variables)）都会按照 tag 形式上报给 InfluxDB。
Review comment:
Actually I'm a little confused about should we add 'zh' or not.
Because accounding to
[Flink+Translation+Specifications](https://cwiki.apache.org/confluence/display/FLINK/Flink+Translation+Specifications)，
' [List of all Variables]({{< ref "zh/docs/ops/metrics" >}}'
looks a little like
> [DataStream API]({% link dev/datastream_api.md %}) 需要修改成中文链接
and it also not like any case like [YARN]
(../ops/deployment/yarn_setup.html)。or [CLI]({% site.baseurl %}/ops/cli.html).
What do you think @RocMarshal ? Seems like the Specifications is a bit
outdated?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
