Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/20816 )

Change subject: [compaction] Add tests to generate high memory rowset compaction
......................................................................


Patch Set 9: Code-Review+2

(1 comment)

http://gerrit.cloudera.org:8080/#/c/20816/6/src/kudu/tablet/compaction-test.cc
File src/kudu/tablet/compaction-test.cc:

http://gerrit.cloudera.org:8080/#/c/20816/6/src/kudu/tablet/compaction-test.cc@637
PS6, Line 637: // into consideration the amount of free memory left and based on
> The problem with setting FLAGS_memory_limit_hard_bytes in the test is that
Yep, with the initialization g_hard_limit it's quite limiting, indeed.

Alternatively, you could separate these tests into their own binary, if it 
makes sense.

It's up to you -- if you are quite confident this approach is good enough with 
updated thresholds, then I'm OK with this approach.  Anyway, if after pushing 
this patch it starts failing from time to time, it will be necessary to 
reconsider this approach.



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I1996558e71c49314c6acf12faf854c796548318c
Gerrit-Change-Number: 20816
Gerrit-PatchSet: 9
Gerrit-Owner: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Ashwani Raina <ara...@cloudera.com>
Gerrit-Reviewer: Attila Bukor <abu...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Comment-Date: Tue, 30 Jan 2024 05:51:34 +0000
Gerrit-HasComments: Yes

Reply via email to