From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <vdavydov.dev@gmail.com>
Date: Tue, 15 May 2018 22:27:15 +0300
From: Vladimir Davydov <vdavydov.dev@gmail.com>
Subject: Re: [PATCH 2/4] vinyl: fix lost key on dump completion
Message-ID: <20180515192715.j3w7t4vhq5r4n6x7@esperanza>
References: <cover.1526392563.git.vdavydov.dev@gmail.com>
 <b6c81d68e19062397a92e78fea62d2964e2bc688.1526392563.git.vdavydov.dev@gmail.com>
 <20180515192043.GB24119@atlas>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <20180515192043.GB24119@atlas>
To: Konstantin Osipov <kostja@tarantool.org>
Cc: tarantool-patches@freelists.org
List-ID: <tarantool-patches.dev.tarantool.org>

On Tue, May 15, 2018 at 10:20:43PM +0300, Konstantin Osipov wrote:
> * Vladimir Davydov <vdavydov.dev@gmail.com> [18/05/15 17:10]:
> 
> I rebased the patch to 1.9, added a comment and pushed the patch.
> 
> Once again, a separate unit test for this edge case would be
> greatly appreciated.

It's rather tricky to write a unit test for this as vy_scheduler depends
on a lot of things. However, this bug is triggered in ~90% cases by
vinyl/select_consistency functional test after the last patch in the
series is applied (and this one is reverted, apparently).

Thanks.