kfaraz commented on PR #18591: URL: https://github.com/apache/druid/pull/18591#issuecomment-3380126203
Thanks for the clarification, @FrankChen021 ! I am just a little apprehensive since the K8s task runner is already pretty buggy. Also, it feels weird to have a config be specified by both static and dynamic means. But I suppose the static config can be thought of as the default value. I haven't gone through the whole PR yet. Will try to do a thorough review today. @GabrielCWT , it would be nice if you could hold off on merging this PR until then. -- 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]
