Hello Adar Dembo, Grant Henke,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/14257
to review the following change.
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
---
M src/kudu/hms/hms_catalog-test.cc
1 file changed, 16 insertions(+), 14 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/57/14257/1
--
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: newchange
Gerrit-Change-Id: I34becd77db193554020ca21171482e6ccd7218d6
Gerrit-Change-Number: 14257
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Grant Henke <[email protected]>
