David Ribeiro Alves has submitted this change and it was merged. 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 Reviewed-on: http://gerrit.cloudera.org:8080/4932 Tested-by: Kudu Jenkins Reviewed-by: Jean-Daniel Cryans <jdcry...@apache.org> --- M src/kudu/tablet/delta_stats.cc M src/kudu/tablet/delta_stats.h M src/kudu/tablet/tablet.proto 3 files changed, 46 insertions(+), 15 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4932 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Idd60f6c1c12803d339f5f8d96c6b089fab21b13f Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: David Ribeiro Alves <dral...@apache.org> Gerrit-Reviewer: Jean-Daniel Cryans <jdcry...@apache.org> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
