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

Change subject: [compaction] Add memory estimation unit test
......................................................................


Patch Set 5:

Btw, I have a patch going out for review, shortly, that does something similar 
by adding two unit tests to exercise the rowset compaction.

The difference is that here we are setting delta size programatically along 
with other parameters. The patch I am working on has two unit tests with and 
without budget restrictions and actual creation of deltas using update 
workload. Feel free to take a look and provide comments.


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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I5397c7e7bf29942a610b0b8203b6918811c49bbb
Gerrit-Change-Number: 20787
Gerrit-PatchSet: 5
Gerrit-Owner: Ádám Bakai <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Ádám Bakai <[email protected]>
Gerrit-Comment-Date: Tue, 19 Dec 2023 12:08:02 +0000
Gerrit-HasComments: No

Reply via email to