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 <[email protected]> --- 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 <[email protected]> Gerrit-Reviewer: David Ribeiro Alves <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
