gyfora commented on code in PR #762: URL: https://github.com/apache/flink-kubernetes-operator/pull/762#discussion_r1478477209
########## flink-kubernetes-operator/src/main/java/org/apache/flink/kubernetes/operator/controller/FlinkResourceContext.java: ########## @@ -65,7 +65,7 @@ public KubernetesJobAutoScalerContext getJobAutoScalerContext() { } private KubernetesJobAutoScalerContext createJobAutoScalerContext() { - Configuration conf = new Configuration(getObserveConfig()); + Configuration conf = new Configuration(getDeployConfig(resource.getSpec())); Review Comment: But you are right that if the user actually changes the memory limits in a new version of the spec then the deploy config would actually make more sense to use. -- 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