Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/8605
to look at the new patch set (#4).
Change subject: tablet: mark delta tracker read-only on error
......................................................................
tablet: mark delta tracker read-only on error
When DeltaTracker::Flush() fails, it leaves a DeltaMemStore in the redo
store list. This is fine for reads because the readpath expects any
DeltaStore to be in that slot, but this is a problem for updates, which
expect only DeltaFileReaders in the list while the compact_flush_lock_
is held.
Before, we would get around this by CHECKing so Flush() could never
return in such a state. This patch introduces a Status to the
DeltaTracker that indicates whether it has experienced such an error.
In order to ensure type-correctness, this Status must be checked
immediately upon entering the critical section of the
compact_flush_lock_.
Change-Id: Ib950048e9cd0929a10714ab1cc2bd829835afced
---
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/delta_tracker.h
2 files changed, 21 insertions(+), 5 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/05/8605/4
--
To view, visit http://gerrit.cloudera.org:8080/8605
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ib950048e9cd0929a10714ab1cc2bd829835afced
Gerrit-Change-Number: 8605
Gerrit-PatchSet: 4
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
