Binglin Chang has posted comments on this change.

Change subject: Fix flakiness in create-table-itest
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/1971/1/src/kudu/integration-tests/create-table-itest.cc
File src/kudu/integration-tests/create-table-itest.cc:

Line 120:       if (inspect_->DoesConsensusMetaExistForTabletOnTS(0, 
tablet_id)) {
I am not familiar with this code, but if we need to check consensus meta, why 
not just check consensus meta?  Does not see ListTabletsOnTS give any more 
information?


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I25ed27a9cf0b4539b6e7a056a2d5c8872e3c1860
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Binglin Chang <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-HasComments: Yes

Reply via email to