Ashwani Raina has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/24578 )

Change subject: Add diff-scan option to return unobservable rows
......................................................................


Patch Set 7:

(1 comment)

Overall looks good to me.
I couldn't complete review today. Planning to do the rest tomorrow.

http://gerrit.cloudera.org:8080/#/c/24578/7//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/24578/7//COMMIT_MSG@14
PS7, Line 14: (INSERT -> eventual DELETE)
question: row-level history is not a requirement here - right ?

For example, INSERT -> DELETE -> INSERT just needs to send back the latest row 
state and not the insert->delete lifecycle information before that.



--
To view, visit http://gerrit.cloudera.org:8080/24578
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I0d0727fb2b89c727c669ec218e3bb8594336359a
Gerrit-Change-Number: 24578
Gerrit-PatchSet: 7
Gerrit-Owner: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Michael Smith <[email protected]>
Gerrit-Comment-Date: Mon, 10 Aug 2026 16:02:03 +0000
Gerrit-HasComments: Yes

Reply via email to