Fix test_secure_socket and remove redundant settings in hedged tests

This commit is contained in:
Pavel Kruglov 2021-04-19 11:46:42 +03:00
parent 3030db9043
commit 8f1b507386
3 changed files with 2 additions and 6 deletions

View File

@ -2,8 +2,6 @@
<yandex> <yandex>
<profiles> <profiles>
<default> <default>
<async_socket_for_remote>1</async_socket_for_remote>
<use_hedged_requests>1</use_hedged_requests>
</default> </default>
</profiles> </profiles>
</yandex> </yandex>

View File

@ -2,8 +2,6 @@
<yandex> <yandex>
<profiles> <profiles>
<default> <default>
<async_socket_for_remote>1</async_socket_for_remote>
<use_hedged_requests>1</use_hedged_requests>
</default> </default>
</profiles> </profiles>
</yandex> </yandex>

View File

@ -64,7 +64,7 @@ def test(started_cluster):
assert end - start < 10 assert end - start < 10
start = time.time() start = time.time()
error = NODES['node1'].query_and_get_error('SELECT * FROM distributed_table settings receive_timeout=5, send_timeout=5, use_hedged_requests=0;') error = NODES['node1'].query_and_get_error('SELECT * FROM distributed_table settings receive_timeout=5, send_timeout=5, use_hedged_requests=0, async_socket_for_remote=1;')
end = time.time() end = time.time()
assert end - start < 10 assert end - start < 10
@ -73,7 +73,7 @@ def test(started_cluster):
assert error.find('DB::ReadBufferFromPocoSocket::nextImpl()') == -1 assert error.find('DB::ReadBufferFromPocoSocket::nextImpl()') == -1
start = time.time() start = time.time()
error = NODES['node1'].query_and_get_error('SELECT * FROM distributed_table settings receive_timeout=5, send_timeout=5;') error = NODES['node1'].query_and_get_error('SELECT * FROM distributed_table settings receive_timeout=5, send_timeout=5, use_hedged_requests=1, async_socket_for_remote=1;')
end = time.time() end = time.time()
assert end - start < 10 assert end - start < 10