Merge pull request #13461 from travisdowns/td-partition-memory-4mb

travisdowns · web-flow · commit b52b0123541a · 2023-09-19T01:45:07.000-03:00
Increase topic_memory_per_partition to 4 MiB
diff --git a/src/v/config/configuration.cc b/src/v/config/configuration.cc
@@ -149,7 +149,7 @@ configuration::configuration()
       "topic_memory_per_partition",
       "Required memory per partition when creating topics",
       {.needs_restart = needs_restart::no, .visibility = visibility::tunable},
-      1_MiB,
+      4_MiB,
       {
         .min = 1,      // Must be nonzero, it's a divisor
         .max = 100_MiB // Rough 'sanity' limit: a machine with 1GB RAM must be
diff --git a/src/v/kafka/server/tests/CMakeLists.txt b/src/v/kafka/server/tests/CMakeLists.txt
@@ -73,6 +73,6 @@ rp_test(
   SOURCES fetch_plan_bench.cc
   LIBRARIES Seastar::seastar_perf_testing Boost::unit_test_framework v::application
   # the args below are just to keep it fast
-  ARGS "-c 1 --duration=1 --runs=1 --memory=1G"
+  ARGS "-c 1 --duration=1 --runs=1 --memory=4G"
   LABELS kafka
 )

Original file line number	Diff line number	Diff line change
`@@ -149,7 +149,7 @@ configuration::configuration()`
`149`	`149`	`"topic_memory_per_partition",`
`150`	`150`	`"Required memory per partition when creating topics",`
`151`	`151`	`{.needs_restart = needs_restart::no, .visibility = visibility::tunable},`
`152`		`- 1_MiB,`
	`152`	`+ 4_MiB,`
`153`	`153`	`{`
`154`	`154`	`.min = 1, // Must be nonzero, it's a divisor`
`155`	`155`	`.max = 100_MiB // Rough 'sanity' limit: a machine with 1GB RAM must be`
Original file line number	Diff line number	Diff line change
`@@ -73,6 +73,6 @@ rp_test(`
`73`	`73`	`SOURCES fetch_plan_bench.cc`
`74`	`74`	`LIBRARIES Seastar::seastar_perf_testing Boost::unit_test_framework v::application`
`75`	`75`	`# the args below are just to keep it fast`
`76`		`- ARGS "-c 1 --duration=1 --runs=1 --memory=1G"`
	`76`	`+ ARGS "-c 1 --duration=1 --runs=1 --memory=4G"`
`77`	`77`	`LABELS kafka`
`78`	`78`	`)`