[tarantool-patches] [PATCH 3/4] vinyl: abort affected transactions when space is removed from cache

Vladimir Davydov vdavydov.dev at gmail.com
Mon Mar 25 11:17:43 MSK 2019


On Mon, Mar 25, 2019 at 08:26:25AM +0300, Георгий Кириченко wrote:
> I definitely dislike introducing a new callback, there are already a lot of 
> dependencies. Also I'm afraid that this could prevent us from the 
> transactional ddl implementation.

I don't see a way to fix this issue without introducing a new callback.
And frankly I don't quite understand your concern. If the new callback
proves to be useless / harmful for transaction DDL, we will remove it -
after all engine API isn't carved in stone.

> We have a space vtab and destroy, why shouldn't we use them?

Because the space becomes invalid before it gets destroyed - ALTER moves
indexes to the new space before writing to WAL.



More information about the Tarantool-patches mailing list