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

Vladimir Davydov vdavydov.dev at gmail.com
Thu Jun 14 11:21:07 MSK 2018


On Wed, Jun 13, 2018 at 08:39:12PM +0300, Konstantin Osipov wrote:
> * 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.

Fixed and pushed to 1.10



More information about the Tarantool-patches mailing list