thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-25 00:52:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix test_extreme_deduplication flakiness

Browse Source 
This is part of **"Why my PR is not merged?"** patch set series.

Since #50107 there one more tunable that can increase the cleanup period
- max_cleanup_delay_period.

Plus, increase the delay for the cleanup thread, otherwise it is too
fragile.

Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
This commit is contained in:
Azat Khuzhin 2023-07-13 13:55:55 +02:00
parent 57c3941fa6
commit 3a54182f77
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
tests/integration/test_extreme_deduplication/configs/conf.d/merge_tree.xml
View File

@ -4,6 +4,7 @@
<replicated_deduplication_window_seconds>1</replicated_deduplication_window_seconds>
<cleanup_delay_period>1</cleanup_delay_period>
<cleanup_delay_period_random_add>0</cleanup_delay_period_random_add>
<max_cleanup_delay_period>1</max_cleanup_delay_period>
<old_parts_lifetime>1</old_parts_lifetime>
</merge_tree>
</clickhouse>

2
tests/integration/test_extreme_deduplication/test.py
View File

@ -50,7 +50,7 @@ def test_deduplication_window_in_seconds(started_cluster):
assert TSV(node.query("SELECT count() FROM simple")) == TSV("2\n")
# wait clean thread
time.sleep(2)
time.sleep(5)
assert (
TSV.toMat(