Hello Adar Dembo, Hao Hao,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/10028
to review the following change.
Change subject: HmsCatalog: fix bug in reconnection backoff logic
......................................................................
HmsCatalog: fix bug in reconnection backoff logic
This fixes an accidentally shadowed Status variable. The effect of this
bug was that the HmsCatalog would not throttle HMS reconnect attempts.
The tests actually took advantage of this, I've had to update them to
try ops multiple times after an HMS restart.
No tests are included, but I've left a TODO about adding tests when
HmsCatalog stats are introduced.
Change-Id: I726283ec8332d16942c82c73492709de4323a43d
---
M src/kudu/hms/hms_catalog-test.cc
M src/kudu/hms/hms_catalog.cc
M src/kudu/integration-tests/master_hms-itest.cc
3 files changed, 29 insertions(+), 14 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/28/10028/1
--
To view, visit http://gerrit.cloudera.org:8080/10028
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I726283ec8332d16942c82c73492709de4323a43d
Gerrit-Change-Number: 10028
Gerrit-PatchSet: 1
Gerrit-Owner: Dan Burkert <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
