deepthi912 opened a new pull request, #14235: URL: https://github.com/apache/pinot/pull/14235
Issue: The environment variable is not setting right with the dynamic.env.config property in controller.conf file. Reference PR: https://github.com/apache/pinot/pull/12307 Root cause: Ideally the dynamic variables should be added once, but BaseControllerStarter & PinotServiceManager(startRole) are both trying to apply dynamic variables and this caused an issue when setting the environment variables from the dynamic.env.config properties. Fix: Made a fix to check for null to ensure that if the environment variable value is already added, it will not add or override again. -- 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
