Jackie-Jiang commented on PR #10105: URL: https://github.com/apache/pinot/pull/10105#issuecomment-1386170909
I discussed with @mayankshriv offline, and we don't need this to be dynamic right now. One issue with the current approach is that we skipped the instance level override and directly applied the cluster level config, which is not aligned with our override rule (instance can override cluster level config). If we want to make cluster level and instance level config dynamically loaded, we can consider adding a rest API to reload the instance level config, so that we still hold the same override sequence. -- 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: commits-unsubscr...@pinot.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: commits-unsubscr...@pinot.apache.org For additional commands, e-mail: commits-h...@pinot.apache.org
