tt0suzy opened a new pull request, #9928: URL: https://github.com/apache/pinot/pull/9928
`bugfix` Context Fixed the flaky test in [9926](https://github.com/apache/pinot/issues/9926). The cause of it is that the test passes a hashmap to `URIUtils.buildURI` and in that function, it uses a for loop to iterate through the map entry and concatenate the parameters into the URI string. However, the order of the the map entry in a hashmap is non-deterministic, so different URI string could be formed even with the same hashmap. Fix In the test, use `LinkedHashMap` instead of `HashMap` so the order of the entry will be deterministic in different runs. -- 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]
