[PATCH] test: fix vinyl/upgrade/fill.lua script

Konstantin Osipov kostja at tarantool.org
Wed Jun 13 20:39:12 MSK 2018


* Vladimir Davydov <vdavydov.dev at gmail.com> [18/06/13 13:42]:
> Since commit 8f63d5d90 ("vinyl: fail transaction immediately if it does
> not fit in memory"), vinyl won't trigger memory dump if the size of
> memory needed by a transaction is greater than the memory limit, instead
> it will fail the transaction immediately. This broke the aforementioned
> script, which relied on this to trigger system-wide memory dump. Fix it
> by reworking the dump trigger logic used by the script: now it tries to
> insert two tuples, box.cfg.vinyl_memory / 2 size each, instead of one.
> 
> Closes #3449


>      while box.stat.vinyl().quota.used > 0 do
>          fiber.sleep(0.1)

Let's settle on 0.01 as standard fiber-loop-wait time in tests.

Otherwise it's ok to push.

>      end
> +    -- Wait for compaction to collect garbage.
> +    while dump_trigger.index.pk:stat().run_count > 1 do
> +        fiber.sleep(0.1)
> +    end
>  end
>  

-- 
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov



More information about the Tarantool-patches mailing list