ClickHouse/programs
Azat Khuzhin a871036361
Fix parallel_reading_from_replicas with clickhouse-bechmark (#34751)
* Use INITIAL_QUERY for clickhouse-benchmark

Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>

* Fix parallel_reading_from_replicas with clickhouse-bechmark

Before it produces the following error:

    $ clickhouse-benchmark --stacktrace -i1 --query "select * from remote('127.1', default.data_mt) limit 10" --allow_experimental_parallel_reading_from_replicas=1 --max_parallel_replicas=3
    Loaded 1 queries.
    Logical error: 'Coordinator for parallel reading from replicas is not initialized'.
    Aborted (core dumped)

Since it uses the same code, i.e RemoteQueryExecutor ->
MultiplexedConnections, which enables coordinator if it was requested
from settings, but it should be done only for non-initial queries, i.e.
when server send connection to another server.

Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>

* Fix 02226_parallel_reading_from_replicas_benchmark for older shellcheck

By shellcheck 0.8 does not complains, while on CI shellcheck 0.7.0 and
it does complains [1]:

    In 02226_parallel_reading_from_replicas_benchmark.sh line 17:
        --allow_experimental_parallel_reading_from_replicas=1
        ^-- SC2191: The = here is literal. To assign by index, use ( [index]=value ) with no spaces. To keep as literal, quote it.

    Did you mean:
        "--allow_experimental_parallel_reading_from_replicas=1"

  [1]: https://s3.amazonaws.com/clickhouse-test-reports/34751/d883af711822faf294c876b017cbf745b1cda1b3/style_check__actions_/shellcheck_output.txt

Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-03-08 16:42:29 +01:00
..
bash-completion Add ability to push down LIMIT for distributed queries 2021-06-09 02:29:50 +03:00
benchmark Fix parallel_reading_from_replicas with clickhouse-bechmark (#34751) 2022-03-08 16:42:29 +01:00
client Merge branch 'master' into random-settings 2022-03-03 13:07:12 +00:00
compressor Move formatBlock to its own file 2021-07-29 16:39:42 +02:00
copier cluster discovery init only with zookeeper 2021-12-27 15:26:08 +03:00
extract-from-config add system.zookeeper_log 2021-07-09 17:05:35 +03:00
format move clickhouse-format code from settings to format.cpp 2022-02-07 13:34:13 +08:00
git-import Rename "common" to "base" 2021-10-02 10:13:14 +03:00
install Add symlinks to keeper 2022-01-27 01:23:18 +03:00
keeper Introduce safeExit() helper (_exit() compatible with TSan) 2022-03-02 22:17:17 +03:00
keeper-converter Rename "common" to "base" 2021-10-02 10:13:14 +03:00
library-bridge Fix formatting error in logging messages 2022-02-01 14:30:04 +03:00
local fix mistake in check-style 2022-03-05 14:22:56 +08:00
obfuscator clang-tidy check performance-noexcept-move-constructor fix 2022-03-02 18:15:27 +00:00
odbc-bridge Fix 2022-02-25 16:04:11 +01:00
server Merge pull request #35068 from ClickHouse/PlayUI_theme_fixes 2022-03-05 02:59:27 +03:00
static-files-disk-uploader static-files-disk-uploader: add mode to create symlinks 2022-02-09 03:50:50 +03:00
clickhouse-split-helper
CMakeLists.txt Remove MAKE_STATIC_LIBRARIES (in favor of USE_STATIC_LIBRARIES) 2022-01-24 17:28:33 +03:00
config_tools.h.in Fix build and add example 2021-08-28 20:35:51 +00:00
embed_binary.S.in Adds a better way to include binary resources 2021-06-09 14:03:30 -07:00
main.cpp Remove arcadia build support 2021-12-11 21:25:23 +03:00