Skip to content

Commit fedb8f4

Browse files
authored
Merge pull request #30058 from bharathv/tx_fix_defaults
config/tx: have reasonable defaults for idempotency/tx configurations.
2 parents 175d8d9 + 7330b45 commit fedb8f4

1 file changed

Lines changed: 3 additions & 8 deletions

File tree

src/v/config/configuration.cc

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -957,14 +957,9 @@ configuration::configuration()
957957
"Maximum number of active producer sessions per shard. Each shard "
958958
"tracks producer IDs using an LRU (Least Recently Used) eviction "
959959
"policy. When the configured limit is exceeded, the least recently "
960-
"used producer IDs are evicted from the cache. IMPORTANT: The default "
961-
"value is unlimited, which can lead to unbounded memory growth and "
962-
"out-of-memory (OOM) crashes in production environments with heavy "
963-
"producer usage, especially when using transactions or idempotent "
964-
"producers. It is strongly recommended to set a reasonable limit in "
965-
"production deployments.",
960+
"used producer IDs are evicted from the cache.",
966961
{.needs_restart = needs_restart::no, .visibility = visibility::tunable},
967-
std::numeric_limits<uint64_t>::max(),
962+
100000,
968963
{.min = 1})
969964
, max_transactions_per_coordinator(
970965
*this,
@@ -976,7 +971,7 @@ configuration::configuration()
976971
"invalid producer epoch or invalid_producer_id_mapping error (depends on "
977972
"the transaction execution phase).",
978973
{.needs_restart = needs_restart::no, .visibility = visibility::tunable},
979-
std::numeric_limits<uint64_t>::max(),
974+
10000,
980975
{.min = 1})
981976
, enable_idempotence(
982977
*this,

0 commit comments

Comments
 (0)