gyfora commented on code in PR #726: URL: https://github.com/apache/flink-kubernetes-operator/pull/726#discussion_r1423504221
########## flink-autoscaler/src/main/java/org/apache/flink/autoscaler/config/AutoScalerOptions.java: ########## @@ -222,6 +222,18 @@ private static ConfigOptions.OptionBuilder autoScalerConfig(String key) { .withDescription( "Processing rate increase threshold for detecting ineffective scaling threshold. 0.1 means if we do not accomplish at least 10% of the desired capacity increase with scaling, the action is marked ineffective."); + public static final ConfigOption<Double> GC_PRESSURE_THRESHOLD = + autoScalerConfig("memory.gc-pressure.threshold") + .doubleType() + .defaultValue(0.3) + .withDescription("Max allowed GC pressure during scaling operations"); + + public static final ConfigOption<Double> HEAP_USAGE_THRESHOLD = + autoScalerConfig("memory.heap-usage.threshold") + .doubleType() + .defaultValue(0.9) Review Comment: I see your point about the heap usage with a large number of TMs and it makes sense. I don't think most jobs would be affected by this issue but we could increase the default threshold to 95% to be on the safe side. Regarding the follow up, please go ahead with the ticket :) and feel free to work on it! -- 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