Hello Mike Percy, Adar Dembo, Todd Lipcon, Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4566 to look at the new patch set (#3). Change subject: [util] Fix a minor bug in AssertEventually() ...................................................................... [util] Fix a minor bug in AssertEventually() We want to be sleeping with a monotonically increasing timeout value in this routine. After 10 attempts, we sleep for 1000 ms(constant value). But this routine was never increasing the 'attempts' thus falling back to sleep always at 1ms intervals making it inefficient. Change-Id: Id34b3461164d96261e4a2d1657244332eb33ad0c --- M src/kudu/util/test_util.cc 1 file changed, 1 insertion(+), 2 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/66/4566/3 -- To view, visit http://gerrit.cloudera.org:8080/4566 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Id34b3461164d96261e4a2d1657244332eb33ad0c Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Dinesh Bhat <din...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>