Andrew Wong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/8441


Change subject: disk failure: pass Statuses instead of failing CHECKs along the 
Flush DMS path
......................................................................

disk failure: pass Statuses instead of failing CHECKs along the Flush DMS path

There are various fatal safety checks along paths surrounding deltas and
the flushing of them. Some of these checks seem redundant given the
layering of these calls; i.e. the fatality of an operation can be
handled at a single point by passing around Statuses.

This patch updates the Flush DMS path to make this the case.

Change-Id: I0241cd4206ce77ef1fb334458b091bc2092f4141
---
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/delta_tracker.h
M src/kudu/tablet/diskrowset-test.cc
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/diskrowset.h
M src/kudu/tablet/rowset.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet_replica_mm_ops.cc
8 files changed, 92 insertions(+), 36 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I0241cd4206ce77ef1fb334458b091bc2092f4141
Gerrit-Change-Number: 8441
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>

Reply via email to