Dan Burkert has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10028 )

Change subject: HmsCatalog: fix bug in reconnection backoff logic
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/10028/1/src/kudu/integration-tests/master_hms-itest.cc
File src/kudu/integration-tests/master_hms-itest.cc:

http://gerrit.cloudera.org:8080/#/c/10028/1/src/kudu/integration-tests/master_hms-itest.cc@279
PS1, Line 279:     // HmsCatalog throttles reconnections, so it's necessary to 
wait out the backoff.
             :     ASSERT_OK(table_alterer->Alter());
             :   });
> Do we have to reconstruct the TableAlterer each time? Couldn't we just call
Done



--
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: comment
Gerrit-Change-Id: I726283ec8332d16942c82c73492709de4323a43d
Gerrit-Change-Number: 10028
Gerrit-PatchSet: 2
Gerrit-Owner: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Dan Burkert <danburk...@apache.org>
Gerrit-Reviewer: Hao Hao <hao....@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Comment-Date: Thu, 12 Apr 2018 19:04:10 +0000
Gerrit-HasComments: Yes

Reply via email to