[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