chore(clickhouse): limit number of concurrent connections for backup#2369
chore(clickhouse): limit number of concurrent connections for backup#2369
Conversation
PR SummaryMedium Risk Overview Reviewed by Cursor Bugbot for commit 6c47e15. Bugbot is set up for automated code reviews on this repo. Configure here. |
There was a problem hiding this comment.
LGTM — straightforward config change adding a connection limit for the ClickHouse backup job.
Extended reasoning...
Overview
Adds a single environment variable CLICKHOUSE_MAX_CONNECTIONS=2 to the clickhouse-backup batch job's Nomad HCL config. The rest of the diff is whitespace alignment.
Security Risks
None. This is a tuning parameter for an internal backup job; no auth, crypto, or permissions code is touched.
Level of Scrutiny
Low. This is a one-line operational config change to a batch job that runs periodically. The variable is a well-known setting for altinity/clickhouse-backup to cap concurrent ClickHouse connections, reducing server pressure during backups.
Other Factors
No bugs were reported by the bug hunting system. The change is consistent with the existing pattern of env-var configuration in this file and has a clear operational motivation.
No description provided.