[PATCH] test: fix vinyl/deferred_delete failure

Vladimir Davydov vdavydov.dev at gmail.com
Fri Mar 15 18:58:12 MSK 2019


On Fri, Mar 15, 2019 at 06:57:51PM +0300, Vladimir Davydov wrote:
> The patch fixes the following test failure:
> 
>  | --- vinyl/deferred_delete.result        Fri Mar 15 18:17:05 2019
>  | +++ vinyl/deferred_delete.reject        Fri Mar 15 18:18:18 2019
>  | @@ -577,7 +577,7 @@
>  |  ...
>  |  sk:stat().rows -- ditto
>  |  ---
>  | -- 5
>  | +- 25
>  |  ...
>  |  s:drop()
>  |  ---
> 
> The failure was introduced by commit 6dd0d2fb7e0e ("vinyl: do not apply
> run_count_per_level to the last level"). Due to the commit compaction of
> the secondary index may happen before compaction of the primary index,
> in which case deferred DELETE statements won't make it to the secondary
> index in time against the test expectation. Fix this by making the first
> run big enough to prevent major compaction from kicking in.
> 
> Follow-up #3657
> Closes #4047
> ---
>  test/vinyl/deferred_delete.result   | 36 ++++++++++++++++++++++--------------
>  test/vinyl/deferred_delete.test.lua | 18 ++++++++++++------
>  2 files changed, 34 insertions(+), 20 deletions(-)

Trivial. Pushed to 2.1.



More information about the Tarantool-patches mailing list