Mike Percy has posted comments on this change.

Change subject: Don't output unobservable rows from the MemRowset
......................................................................


Patch Set 13:

> 
 > DRS 1:
 > undo: del @ 2
 > base: v1
 > redo delta: del @3
 > 
 > DRS 2:
 > base: v2
 > 
 > 
 > ----------------
 > flush and compact
 > ---------------
 > 
 > ?

I think in this case you would still have a del UNDO @ 3 in DRS 2, so you could 
reason that at the same TS a del UNDO in a higher RS comes after a del REDO in 
a lower RS. Right?

-- 
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: 13
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

Reply via email to