You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org> on 2016/11/08 08:49:28 UTC
[kudu-CR] Account for REINSERTs in delta stats
Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/4932
to look at the new patch set (#2).
Change subject: Account for REINSERTs in delta stats
......................................................................
Account for REINSERTs in delta stats
This makes DeltaStats also account for REINSERTs. This is not
actually used outside of tests, but seems like it would be
silly to not count this type of delta. In the future it might
be useful for selecting undo deltas for minor delta compaction.
The protobuf field is optional to preserve data format
compatibility.
Change-Id: Idd60f6c1c12803d339f5f8d96c6b089fab21b13f
---
M src/kudu/tablet/delta_stats.cc
M src/kudu/tablet/delta_stats.h
M src/kudu/tablet/tablet.proto
3 files changed, 43 insertions(+), 11 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/32/4932/2
--
To view, visit http://gerrit.cloudera.org:8080/4932
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Idd60f6c1c12803d339f5f8d96c6b089fab21b13f
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: David Ribeiro Alves <dr...@apache.org>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot