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