tsreaper commented on a change in pull request #16683:
URL: https://github.com/apache/flink/pull/16683#discussion_r692065256
##########
File path:
docs/layouts/shortcodes/generated/prometheus_push_gateway_reporter_configuration.html
##########
@@ -48,7 +48,7 @@
<td><h5>randomJobNameSuffix</h5></td>
<td style="word-wrap: break-word;">true</td>
<td>Boolean</td>
- <td>Specifies whether a random suffix should be appended to the
job name.</td>
+ <td>Specifies whether a random suffix should be appended to the
job name. Job at least have two container(JobManager and TaskManager), enabled
it can diff each other.</td>
Review comment:
This description is somewhat incorrect. This config adds different
random suffix for different reporter instances, but it is not necessary that
each task manager has its own metrics reporter (although for most of the time
it is the case). For example a mini cluster only has one metrics reporter (see
`MiniCluster#start`).
--
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]
