[tarantool-patches] Re: [PATCH 24/25] Introduce _vinyl_deferred_delete system space

[Konstantin Osipov]

* Vladimir Davydov <vdavydov.dev at gmail.com> [18/08/01 18:14]:
> On Tue, Jul 31, 2018 at 11:54:27PM +0300, Konstantin Osipov wrote:
> > * Vladimir Davydov <vdavydov.dev at gmail.com> [18/07/27 16:55]:
> > > The space is a blackhole. It will be used for writing deferred DELETE
> > > statements generated by vinyl compaction tasks to WAL so that we can
> > > recover deferred DELETEs that hadn't been dumped to disk before the
> > > server was restarted.
> > 
> > Why do you actually need a proto space for it? Please feel free to
> > put blackhole engine first in the list of engines.
> 
> You mean making this space purely virtual? No record in _space, no info
> in snap file, just struct space in the cache?

No, I mean no changes in sc_* methods, only a record in _space?

> Hmm, I guess we can do that. However, I think that it's kinda weird
> to have a space that can be written to, but which doesn't exist in the
> data dictionary. This may also result in some anomalies, like the user
> can grant access to it although it doesn't actually exist in _space
> (yeah, this is pointless, I know, but still).
> 
> Is there any reason to do that? Or do you mean something different by
> 'proto space'?

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