Hello Mike Percy, Adar Dembo,

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

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

to review the following change.

Change subject: ts_tablet_manager: set status message after failed tablet copy
......................................................................

ts_tablet_manager: set status message after failed tablet copy

I noticed in a test cluster that a tablet copy had failed, and ksck was
reporting the state as:

  b314b10216d74c07ab9da974671137db (ve0136.halxg.cloudera.com:7050): bad state
    State:       NOT_STARTED
    Data state:  TABLET_DATA_TOMBSTONED
    Last status: TabletCopy: Downloading block 4611685870251245460 (2342/33956)

This was rather confusing, because it claimed it was downloading a
block, and also claimed to be tombstoned at the same time (rather than
being in the COPYING state).

This fixes the failure case to set the status message properly.

Change-Id: I36ecd45dacec4ea329a0f0dd236e52bc603f9b93
---
M src/kudu/integration-tests/delete_table-test.cc
M src/kudu/tserver/ts_tablet_manager.cc
M src/kudu/tserver/ts_tablet_manager.h
3 files changed, 17 insertions(+), 9 deletions(-)


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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I36ecd45dacec4ea329a0f0dd236e52bc603f9b93
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>

Reply via email to