Fly-Style commented on code in PR #19286: URL: https://github.com/apache/druid/pull/19286#discussion_r3121994671
########## indexing-service/src/main/java/org/apache/druid/indexing/seekablestream/supervisor/autoscaler/CostBasedAutoScaler.java: ########## Review Comment: @jtuglu1, the config always was a source of truth, implying there is no sense in making a computation for a case that never happens. > My issue with this is that we are removing observability into the case where the min/max task count is indeed misconfigured relative to what the autoscaler thinks is the optimal task count. That's fair, but there was no intention to make autoscaler smarter than it is. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
