You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Adar Dembo (Code Review)" <ge...@cloudera.org> on 2016/02/18 02:16:26 UTC

[kudu-CR] master: fix usage of iterators in TableInfo::GetTabletsInRange

Hello Todd Lipcon,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/2217

to review the following change.

Change subject: master: fix usage of iterators in TableInfo::GetTabletsInRange
......................................................................

master: fix usage of iterators in TableInfo::GetTabletsInRange

The misuse is only fatal in a verrrrry specific set of circumstances which
I've managed to distill in the new integration test. Without the patch
applied, the test fails after five seconds every time. With it, it doesn't
appear to fail at all.

Change-Id: I93fb869ca2c7849c43e0167d5eb7eaf93c27d48d
---
M src/kudu/integration-tests/create-table-itest.cc
M src/kudu/master/catalog_manager.cc
2 files changed, 96 insertions(+), 1 deletion(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/17/2217/1
-- 
To view, visit http://gerrit.cloudera.org:8080/2217
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I93fb869ca2c7849c43e0167d5eb7eaf93c27d48d
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>