[Tarantool-patches] [PATCH vshard 3/7] test: disable router discovery for some tests

Oleg Babin olegrok at tarantool.org
Fri May 1 20:00:28 MSK 2020


Hi. Thanks for the patch. LGTM. But I have a question.

On 01/05/2020 03:16, Vladislav Shpilevoy wrote:
> There are tests, which are not related to discovery at all, and
> yet they suffer from unrelated changes, when something is modified
> in the discovery algorithm. For example, it highly affects the
> test about request retries and automatic timeouts.
> 
> Now the discovery is disabled for such tests, since it becomes
> easily possible to do with the new router's configuration option.
> ---
> -_ = test_run:cmd("start server router_1")
> +-- Discovery algorithm changes sometimes and should not affect the
> +-- exponential timeout test.
> +_ = test_run:cmd("start server router_1 with args='discovery_disable'")
>   ---
>   ...
>   _ = test_run:switch('router_1')
> @@ -49,13 +51,13 @@ util.collect_timeouts(rs1)
>       ok: 0
>       timeout: 0.5
>     - fail: 0
> -    ok: 1
> +    ok: 0
>       timeout: 0.5
>   ...
>   util.collect_timeouts(rs2)
>   ---
>   - - fail: 0
> -    ok: 1
> +    ok: 0
>       timeout: 0.5
>     - fail: 0
>       ok: 0
> @@ -74,7 +76,7 @@ util.collect_timeouts(rs1)
>       ok: 0
>       timeout: 0.5
>     - fail: 0
> -    ok: 9
> +    ok: 8
>       timeout: 0.5
>   ...
>   _ = rs1:callrw('echo')
> @@ -86,7 +88,7 @@ util.collect_timeouts(rs1)
>       ok: 0
>       timeout: 0.5
>     - fail: 0
> -    ok: 1
> +    ok: 9
>       timeout: 0.5
>   ...

Sorry for maybe quite stupid question, but why does it quite 
significantly increase?
```
net_sequential_ok - count of sequential success requests to the replica
```


More information about the Tarantool-patches mailing list