Hello David Ribeiro Alves, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5920
to look at the new patch set (#4).
Change subject: compaction: Flush tablet metadata before updating stores
......................................................................
compaction: Flush tablet metadata before updating stores
We should always flush the rowset metadata before making changes visible
to the delta tracker. In this case, because we are just doing a
compaction, it is likely not a crash consistency issue, however we
should always follow the same order of operations when operating on
tablet metadata.
Also, add LogPrefix() for the log lines printed from the delta tracker.
Change-Id: I958f4744f1b95c6a63e8c9105bd1c7552e43e6b9
---
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/delta_tracker.h
M src/kudu/tablet/diskrowset.cc
3 files changed, 61 insertions(+), 40 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/20/5920/4
--
To view, visit http://gerrit.cloudera.org:8080/5920
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I958f4744f1b95c6a63e8c9105bd1c7552e43e6b9
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: David Ribeiro Alves <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <[email protected]>
