Adar Dembo has posted comments on this change.

Change subject: ts_itest-base.h: wait for bootstrapping to finish when waiting 
for replicas
......................................................................


Patch Set 1: Verified+1

Found a couple interesting things while debugging the Java test failure.
1) It's due to KUDU-1473, fixed in another patch in this series.
2) Still not quite sure why Java tests trigger it so much. I think it's because 
TestKuduTable hammers the cluster with a combination of CreateTable() and 
GetTableLocations() calls. I'll try to reproduce this in a C++ test.
3) The surefire rerunning functionality doesn't work properly on this test, 
which expects a table count of 6 (5 from testGetLocations and 1 from 
testAlterTable), but when rerunning, only the 5 tables from testGetLocations 
are actually created. We should address this.

-- 
To view, visit http://gerrit.cloudera.org:8080/3308
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: I116e0bd8ec9d7abbe830d1d0ea4e35465d990a28
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: No

Reply via email to