alpinegizmo commented on code in PR #20598:
URL: https://github.com/apache/flink/pull/20598#discussion_r947101942


##########
docs/content/docs/deployment/metric_reporters.md:
##########
@@ -245,6 +245,8 @@ Type: push/tags
 Note any variables in Flink metrics, such as `<host>`, `<job_name>`, 
`<tm_id>`, `<subtask_index>`, `<task_name>`, and `<operator_name>`,
 will be sent to Datadog as tags. Tags will look like `host:localhost` and 
`job_name:myjobname`.
 
+<span class="label label-danger">Note</span> For legacy reasons the reporter 
uses _both_ the metric identifier _and_ tags. This redundancy can be avoided by 
enabling `useLogicalIdentifier`.

Review Comment:
   If the default is true, then this wording is a bit misleading. 



-- 
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: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to