From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Date: Tue, 27 Nov 2018 12:56:48 +0300 From: Vladimir Davydov Subject: Re: [PATCH 2/6] engine: pass vclock instead of lsn to collect_garbage callback Message-ID: <20181127095648.545on57gxouzfb4p@esperanza> References: <176d11be84dc464db2467aa34f4b2d42b6e70350.1543152574.git.vdavydov.dev@gmail.com> <20181126174147.GC7839@chai> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <20181126174147.GC7839@chai> To: Konstantin Osipov Cc: tarantool-patches@freelists.org List-ID: On Mon, Nov 26, 2018 at 08:41:47PM +0300, Konstantin Osipov wrote: > * Vladimir Davydov [18/11/26 10:27]: > > First, this is consistent with other engine callbacks, such as > > checkpoint or backup. > > > > Second, a vclock can be used as a search key in a vclock set, > > which in turn can make code more straightforward, e.g. look how > > this patch simplifies vy_log_prev_checkpoint(). > > OK to push. Pushed to 2.1