[Tarantool-patches] [PATCH v3 03/13] vinyl: rename tx_manager -> vy_tx_manager

Aleksandr Lyapunov alyapunov at tarantool.org
Thu Jul 16 11:17:33 MSK 2020


I'm not sure.. Actually I believe one there will be only one 
cross-engine transaction engine..
That's why I made it outstanding. But actually I'm note sure it's possible.
What do you think is better? use memtx specific name and if/when it'll 
be possible
to used in both engines - rename it OR use general name and rename to memtx
specific if/when it becomes obvious that it's not possible?

On 15.07.2020 19:04, Nikita Pettik wrote:
> On 15 Jul 16:55, Aleksandr Lyapunov wrote:
>> Apart from other vinyl objects that are named with "vy_" prefix,
>> its transaction manager (tx_manager) have no such prefix.
>> It should have in order to avoid conflicts with global tx manager.
>>
>> Needed for #4897
>> ---
> LGTM. The only concern I have now - mb we'd better call
> new TX manager in memtx not tx_manager, but mem_tx_manager or
> memtx_tx_manager? I mean tx_ prefix is more general than memtx_..
>


More information about the Tarantool-patches mailing list