Skip to content

Commit b707b75

Browse files
committed
fix(hosting): bound part_log, cap query threads
part_log was not implicated in either report behind this change and is not on ClickHouse's sub-16GB disable list, but it is the merge history you need to diagnose the next occurrence. Measured at 0.18 KiB per part event under insert churn - 10x cheaper than text_log over the same window - so a TTL bounds it rather than removing it. max_threads is the one sub-16GB recommendation the stack was still missing: it resolved to auto(4) and queries really did use 4 threads, so concurrent_threads_soft_limit_num was not covering it.
1 parent 371a0a8 commit b707b75

2 files changed

Lines changed: 4 additions & 1 deletion

File tree

hosting/docker/clickhouse/override.xml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@
1313
<!-- ClickHouse's own telemetry, too heavy for the recommended machine size -->
1414
<metric_log remove="1"/>
1515
<asynchronous_metric_log remove="1"/>
16-
<part_log remove="1"/>
1716
<trace_log remove="1"/>
1817
<query_thread_log remove="1"/>
1918
<session_log remove="1"/>
@@ -28,6 +27,9 @@
2827
<query_views_log remove="1"/>
2928

3029
<!-- A config-level ttl survives log-table recreation, unlike ALTER ... MODIFY TTL -->
30+
<part_log>
31+
<ttl>event_date + INTERVAL 7 DAY DELETE</ttl>
32+
</part_log>
3133
<query_log>
3234
<ttl>event_date + INTERVAL 7 DAY DELETE</ttl>
3335
</query_log>

hosting/docker/clickhouse/users-override.xml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,7 @@
11
<clickhouse>
22
<profiles>
33
<default>
4+
<max_threads>1</max_threads>
45
<max_block_size>8192</max_block_size>
56
<max_download_threads>1</max_download_threads>
67
<input_format_parallel_parsing>0</input_format_parallel_parsing>

0 commit comments

Comments
 (0)