KarmaGYZ commented on a change in pull request #7402: [hotfix][docs] Fulfill 
some empty description in Config Doc
URL: https://github.com/apache/flink/pull/7402#discussion_r246380092
 
 

 ##########
 File path: 
flink-core/src/main/java/org/apache/flink/configuration/MetricOptions.java
 ##########
 @@ -135,14 +139,18 @@
         */
        public static final ConfigOption<Boolean> SYSTEM_RESOURCE_METRICS =
                key("metrics.system-resource")
-                       .defaultValue(false);
+                       .defaultValue(false)
+                       .withDescription("Flag indicating whether Flink should 
report system resource metrics such as machine's CPU," +
+                               " memory or network usage.");
        /**
         * Interval between probing of system resource metrics specified in 
milliseconds. Has an effect only when
         * {@link #SYSTEM_RESOURCE_METRICS} is enabled.
         */
        public static final ConfigOption<Long> 
SYSTEM_RESOURCE_METRICS_PROBING_INTERVAL =
                key("metrics.system-resource-probing-interval")
-                       .defaultValue(5000L);
+                       .defaultValue(5000L)
+                       .withDescription("Interval between probing of system 
resource metrics specified in milliseconds. Has an effect" +
+                               " only when SYSTEM_RESOURCE_METRICS is 
enabled");
 
 Review comment:
   Good catch! Revised.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services

Reply via email to