You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@kudu.apache.org by al...@apache.org on 2025/02/03 05:39:40 UTC
(kudu) 01/03: [test] Address leftover comment from commit id d405ad3
This is an automated email from the ASF dual-hosted git repository.
alexey pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/kudu.git
commit cf2fcb317e028e1f35d16639937c046173792caa
Author: Ashwani Raina <ar...@cloudera.com>
AuthorDate: Wed Jan 29 13:28:42 2025 +0530
[test] Address leftover comment from commit id d405ad3
Trivial change to make compaction_mem_usage_approx compile time constant.
Addresses the following leftover comment:
https://gerrit.cloudera.org/c/22079/16/src/kudu/tablet/compaction-highmem-test.cc#244
Change-Id: Icc49acbfd713f5f9f9cf6635b87825f2c1622299
Reviewed-on: http://gerrit.cloudera.org:8080/22416
Reviewed-by: Zoltan Martonka <zm...@cloudera.com>
Reviewed-by: Attila Bukor <ab...@apache.org>
Tested-by: Attila Bukor <ab...@apache.org>
---
src/kudu/tablet/compaction-highmem-test.cc | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/src/kudu/tablet/compaction-highmem-test.cc b/src/kudu/tablet/compaction-highmem-test.cc
index 7bb0b3e1b..63301898b 100644
--- a/src/kudu/tablet/compaction-highmem-test.cc
+++ b/src/kudu/tablet/compaction-highmem-test.cc
@@ -241,7 +241,7 @@ TEST_F(TestHighMemCompaction, TestMajorCompactionMemoryPressure) {
// Approximate memory consumed by delta compaction during the course of this test.
// Total consumption would always exceed this usage. Since we want to be
// certain that warning threshold limit is crossed, this value is kept low.
- const int64_t compaction_mem_usage_approx = 3 * 1024 * 1024;
+ constexpr int64_t compaction_mem_usage_approx = 3 * 1024 * 1024;
// Set appropriate flags to ensure memory threshold checks fail.
FLAGS_memory_limit_compact_usage_warn_threshold_percentage =