Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/10929 )
Change subject: memrowset: support iteration with include_deleted_rows ...................................................................... memrowset: support iteration with include_deleted_rows This is another piece in the incremental backup puzzle. The idea is that when taking an incremental backup, the scan will include rows that have been deleted. The backup will figure out which rows were deleted by including a special "is deleted" virtual column in the projection. This commit introduces a new iterator option that can be used to include deleted rows, and uses it in the MemRowSet. Change-Id: Ie018043518b437ecc719cf9f87b2c5eea560c9a1 Reviewed-on: http://gerrit.cloudera.org:8080/10929 Reviewed-by: Todd Lipcon <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Mike Percy <[email protected]> --- M src/kudu/tablet/memrowset-test.cc M src/kudu/tablet/memrowset.cc M src/kudu/tablet/rowset.cc M src/kudu/tablet/rowset.h 4 files changed, 80 insertions(+), 16 deletions(-) Approvals: Todd Lipcon: Looks good to me, but someone else must approve Kudu Jenkins: Verified Mike Percy: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/10929 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie018043518b437ecc719cf9f87b2c5eea560c9a1 Gerrit-Change-Number: 10929 Gerrit-PatchSet: 7 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Grant Henke <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]>
