jadami10 opened a new pull request, #13214: URL: https://github.com/apache/pinot/pull/13214
This is a bugfix to enforce unique, relaxed env variables when loading the PinotConfiguration related to https://github.com/apache/pinot/pull/12307. Our build system adds the same env variable in 2 formats, and the conversion from _ to . formatting in PinotConfiguration::relaxEnvVarName causes an IllegalStateException because the variables end up with the same name. This change effectively enforces we always take the alphabetically sorted first value instead. This has no impact on existing users since they would have seen exceptions if they were doing this. -- 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
