pthariensflame opened a new pull request #2719: URL: https://github.com/apache/hive/pull/2719
### What changes were proposed in this pull request? The test method `org.apache.hadoop.hive.metastore.utils.TestMetaStoreServerUtils.testGetPartitionspecsGroupedBySDonePartitionCombined` creates a `List<PartitionSpec>` that was order-unstable under [NonDex](https://github.com/TestingResearchIllinois/NonDex); sorting it ensures that the partitions are in a known order while preserving the intent of the test. ### Why are the changes needed? This test passes as is, but if the implementation of `MetaStoreServerUtils.getPartitionspecsGroupedByStorageDescriptor` were to ever change order, even implicitly through a changed dependency, then the test would break. This PR fixes the test to no longer depend on the order of partitions returned. ### Does this PR introduce _any_ user-facing change? No; this only alters one test, that was already passing normally, to additionally pass if the latent order-instability ends up happening in practice. ### How was this patch tested? The test itself was run before alteration, both normally (to ensure it already passed) and with NonDex (to confirm the flakiness); it was then run again in both ways after alteration, at which point both normal and NonDex runs passed, including over 150 different seeds for the latter. -- 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]
