kfaraz commented on PR #17903: URL: https://github.com/apache/druid/pull/17903#issuecomment-2863341504
> In this case, the proportional scaling config parameters' meanings differ from what their documented use is with the threshold-based scaler. I'm wondering if this is something we'd want to incorporate into the current scaler, or perhaps create a new one altogether. Thanks for pointing this out, @jtuglu-netflix ! In general, I feel that it is always better to improve the logic and update the documentation accordingly rather than stick to a poorer and buggy implementation. However, I will try to take another look at the docs + code and double check if this can have any adverse and undesirable impact on existing supervisor specs. -- 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]
