konstantinb commented on PR #6729: URL: https://github.com/apache/hive/pull/6729#issuecomment-5443032378
@abstractdog @architjainjain could you take a look when you have a chance? This fixes the flaky `UnnecessaryStubbingException` failures in `TestYarnQueueMetricsCollector`, introduced with the HIVE-27126 test suite (#6501), that have started hitting unrelated precommit runs — e.g. [PR 6676 run 6](https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-6676/6/tests) and [run 7](https://ci.hive.apache.org/blue/organizations/jenkins/hive-precommit/detail/PR-6676/7/tests). Each run flagged a different test in the class (`testMetricsCollectionWithNullQueueInfo`, then `testExceptionDuringCollection`) — whichever strict stub loses the race with `shutdown()` gets reported; all of them are covered here. The PR description has a one-command deterministic repro; all checks on this PR are green. -- 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]
