jadami10 opened a new pull request, #13213: URL: https://github.com/apache/pinot/pull/13213
This is a `bugfix` to enforce unique, relaxed env variables when loading the PinotConfiguration related to #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
