[tarantool-patches] Re: [PATCH 0/3] Merge struct sql_txn into struct txn/savepoint
Konstantin Osipov
kostja at tarantool.org
Wed Aug 7 18:26:41 MSK 2019
* Nikita Pettik <korablev at tarantool.org> [19/08/07 18:19]:
> Branch: https://github.com/tarantool/tarantool/tree/np/move-sql-structs-from-txn
>
> As a one of final steps of merging SQL and NoSQL codebases, it is
> required to squash struct sql_txn and struct txn/struct txn_savepoint.
> struct sql_txn was needed to operate on named savepoints. Hence, it
> contains name of savepoint and a link to next savepoint. This patch-set
> adds optional name of savepoint to struct txn_savepoint and orginizes
> txn_savepoints into list. Head of list is held in stuct txn. Iterating
> over list allows to find savepoint by its name. Finally, after this
> procedure is completed, we can remove struct sql_txn and struct Savepoint.
Thanks for the patch, overall LGTM (but I did not look carefully).
--
Konstantin Osipov, Moscow, Russia
More information about the Tarantool-patches
mailing list