[Tarantool-patches] [PATCH v4 10/12] txm: use new tx manager in memtx
Nikita Pettik
korablev at tarantool.org
Wed Sep 23 13:26:06 MSK 2020
On 22 Sep 20:53, Aleksandr Lyapunov wrote:
> Hi, thanks for the review.
> Almost all if fixed, see my comment below:
>
> On 15.09.2020 20:59, Nikita Pettik wrote:
> >
> > > diff --git a/src/box/vinyl.c b/src/box/vinyl.c
> > > index aa6e50f..cee39c5 100644
> > > --- a/src/box/vinyl.c
> > > +++ b/src/box/vinyl.c
> > > @@ -1076,7 +1076,7 @@ vinyl_space_check_format(struct space *space, struct tuple_format *format)
> > > return -1;
> > > /* See the comment in vinyl_space_build_index(). */
> > > - txn_can_yield(txn, true);
> > > + bool could_yield = txn_can_yield(txn, true);
> > Don't get why vinyl related functions should be involved in this change..
> > The rest is OK.
> These vinyl functions are called in memtx transaction that performs alter of
> a vinyl space..
> In the code vinyl changes behavior of the memtx transaction.
Ok, get it, thx.
More information about the Tarantool-patches
mailing list