Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/14257 )
Change subject: deflake HmsCatalogTest.TestMetastoreUuid ...................................................................... deflake HmsCatalogTest.TestMetastoreUuid In the test, it's possible that the first time we get the UUID, we don't actually get anything, e.g., because the HMS is still starting up. If that happens, the first call will respond with a NotSupported error, even if the HMS supports UUIDs. The later fetch of the UUID may successfully return a UUID and then hit an assertion failure. When running the test with a version of Kudu that uses Hive 3.1, the test failed 7/20 times. With this patch, the test passed 100/100 times. Change-Id: I34becd77db193554020ca21171482e6ccd7218d6 Reviewed-on: http://gerrit.cloudera.org:8080/14257 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Andrew Wong <[email protected]> --- M src/kudu/hms/hms_catalog-test.cc 1 file changed, 16 insertions(+), 14 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Andrew Wong: Verified -- To view, visit http://gerrit.cloudera.org:8080/14257 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I34becd77db193554020ca21171482e6ccd7218d6 Gerrit-Change-Number: 14257 Gerrit-PatchSet: 2 Gerrit-Owner: Andrew Wong <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
