thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-09-20 16:50:48 +00:00
Code Issues Packages Projects Releases Wiki Activity

Fix 01085_max_distributed_connections flackiness

Browse Source 
1.8 seconds is too small delay, which eventually can lead to
false-positive if the machine will be slow enough.

Increase number of remote streams and the delay, to avoid
false-positive.
This commit is contained in:
Azat Khuzhin 2020-08-28 02:39:20 +03:00
parent 25eb53636d
commit 07923cd7cd
2 changed files with 11 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
tests/queries/0_stateless/01085_max_distributed_connections.reference
View File

@ -1,2 +1,9 @@
0
0
0
0
0
0
0
0
0

8
tests/queries/0_stateless/01085_max_distributed_connections.sh
View File

@ -4,10 +4,10 @@ CURDIR=$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)
. "$CURDIR"/../shell_config.sh
opts=(
--max_distributed_connections 2
--max_distributed_connections 9
--max_threads 1
--query "SELECT sleepEachRow(1) FROM remote('127.{2,3}', system.one)"
--query "SELECT sleepEachRow(1) FROM remote('127.{2..10}', system.one)"
)
# 1.8 less then 2 seconds, but long enough to cover possible load peaks
# 5 less then 9 seconds (9 streams), but long enough to cover possible load peaks
# "$@" left to pass manual options (like --experimental_use_processors 0) during manual testing
timeout 1.8s ${CLICKHOUSE_CLIENT} "${opts[@]}" "$@"
timeout 5s ${CLICKHOUSE_CLIENT} "${opts[@]}" "$@"