mirror of
https://github.com/ClickHouse/ClickHouse.git
synced 2024-12-04 13:32:13 +00:00
a4f765cae7
* Configure keeper for perf tests Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com> * Improve performance of SELECTs with active mutations getAlterMutationCommandsForPart() can be a hot path for query execution when there are pending mutations. - LOG_TEST - it is not only check one bool, but actually a bunch of atomics as well. - Return std::vector over std::map (map is not required there) - no changes in performance. - Copy only RENAME_COLUMN (since only this mutation is required by AlterConversions). And here are results: run|result -|- SELECT w/o ALTER|queries: 1565, QPS: 355.259, RPS: 355.259 SELECT w/ ALTER unpatched|queries: 2099, QPS: 220.623, RPS: 220.623 SELECT w/ ALTER and w/o LOG_TEST|queries: 2730, QPS: 235.859, RPS: 235.859 SELECT w/ ALTER and w/o LOG_TEST and w/ RENAME_COLUMN only|queries: 2995, QPS: 290.982, RPS: 290.982 But there are still room for improvements, at least MergeTree engines could implement getStorageSnapshotForQuery(). Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com> * Add AlterConversions::supportsMutationCommandType(), flatten vector<vector<MutationCommand>> * Work around what appears to be a clang static analysis bug --------- Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com> Co-authored-by: Michael Kolupaev <michael.kolupaev@clickhouse.com>
25 lines
705 B
XML
25 lines
705 B
XML
<clickhouse>
|
|
<http_port remove="remove"/>
|
|
<mysql_port remove="remove"/>
|
|
<postgresql_port remove="remove"/>
|
|
<tcp_with_proxy_port remove="remove"/>
|
|
<listen_host>::</listen_host>
|
|
|
|
<logger>
|
|
<console>true</console>
|
|
</logger>
|
|
|
|
<text_log remove="remove"/>
|
|
<crash_log remove="remove"/>
|
|
<query_views_log remove="remove"/>
|
|
<part_log remove="remove"/>
|
|
<opentelemetry_span_log remove="remove"/>
|
|
<session_log remove="remove"/>
|
|
|
|
<uncompressed_cache_size>1000000000</uncompressed_cache_size>
|
|
|
|
<asynchronous_metrics_update_period_s>10</asynchronous_metrics_update_period_s>
|
|
|
|
<remap_executable replace="replace">true</remap_executable>
|
|
</clickhouse>
|