rionmonster commented on code in PR #27744:
URL: https://github.com/apache/flink/pull/27744#discussion_r2902536451
##########
flink-metrics/flink-metrics-prometheus/src/main/java/org/apache/flink/metrics/prometheus/PrometheusPushGatewayReporterOptions.java:
##########
@@ -45,13 +45,6 @@ public class PrometheusPushGatewayReporterOptions {
.defaultValue("")
.withDescription("The job name under which metrics will be
pushed");
- public static final ConfigOption<Boolean> RANDOM_JOB_NAME_SUFFIX =
Review Comment:
Are we sure we want to remove this configuration entirely? I’d imagine there
are existing jobs and/or users already relying on it. This would also require
updating the associated documentation and, at a minimum, feels like something
that should be deprecated before being removed.
--
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]
