Jackie-Jiang commented on PR #10105: URL: https://github.com/apache/pinot/pull/10105#issuecomment-1381067254
> @Jackie-Jiang the PR is still required so that if cluster config is changed post start, we can use the new config for groovy. IMO we don't need to reflect the config change dynamically. All the cluster level configs are read during instance start, and we should just follow the same way for this config. If user wants to enable groovy, they can add it into the cluster config, then restart the controllers to pick it up -- 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]
