Todd Lipcon has posted comments on this change. Change subject: Don't output unobservable rows from the MemRowset ......................................................................
Patch Set 11: I'm still a little worried about the "manufactured" nature of this test. Is this scenario produced by any higher level test? eg the fuzz tests, etc? Do we have coverage for the other similar scenario: initial state -------------- DRS 1: undo: delete @2 base: v1 MRS empty ------- txn @3: delete, insert v2 ---------- DRS 1: undo: del @ 2 base: v1 redo delta: del @3 MRS: undo: del @3 base: v2 ----- flush MRS, AHM = 5 ------- DRS 1: undo: del @ 2 base: v1 redo delta: del @3 DRS 2: base: v2 ---------------- flush and compact --------------- ? -- To view, visit http://gerrit.cloudera.org:8080/4994 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ieab52a1aa68494218f91f3acd31ef8ddf352bd57 Gerrit-PatchSet: 11 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]> Gerrit-HasComments: No
