clarax commented on PR #685: URL: https://github.com/apache/flink-kubernetes-operator/pull/685#issuecomment-1776429332
> Another thing we have been discussing with @1996fanrui > > The interval config should probably be renamed `scaling.report.interval->scaling.event.interval` this way we can use it generally in the future for autoscaler triggered events. > > We should also make sure that the simple `handleEvent` method also respects the interval if specified. And we should probably use the interval also for ineffective scaling events. I know that some of these changes are not directly related to this PR but it may be better to clean it up so we leave it in a good state afterwards. Resolved all requested changes. -- 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
