Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/13534 )
Change subject: KUDU-2809 (2/6): skip unobservable rows when iterating an MRS ...................................................................... KUDU-2809 (2/6): skip unobservable rows when iterating an MRS An unobservable row is one whose entire lifespan is contained within the time range of a diff scan. Such rows should be hidden from the diff scan results as they represent "changes" that the client wouldn't expect to see. This patch adds such a heuristic to the MRS iterator. It's relatively straight-forward because the MRS only stores REDOs and always remembers its rows' insertion timestamps. Delta stores are more complicated; a patch for handling unobservable rows for them will follow. Change-Id: I801991260749d1f810540cb32ec84c3ea6a02160 Reviewed-on: http://gerrit.cloudera.org:8080/13534 Reviewed-by: Mike Percy <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/tablet/memrowset-test.cc M src/kudu/tablet/memrowset.cc M src/kudu/tablet/memrowset.h M src/kudu/tablet/tablet-test.cc 4 files changed, 132 insertions(+), 11 deletions(-) Approvals: Mike Percy: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/13534 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I801991260749d1f810540cb32ec84c3ea6a02160 Gerrit-Change-Number: 13534 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 (120) Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]>
