[Tarantool-patches] [PATCH 2/5] [tosquash] applier: remove unnecessary fiber name check
Serge Petrenko
sergepetrenko at tarantool.org
Sun Jul 5 11:40:36 MSK 2020
03.07.2020 02:40, Vladislav Shpilevoy пишет:
> This was an artifact from a first version of the patch.
> Of course it is not valid, because final commit of this
> transaction can be done by any fiber.
Hi! Thanks for the patch! LGTM.
> ---
> src/box/applier.cc | 1 -
> 1 file changed, 1 deletion(-)
>
> diff --git a/src/box/applier.cc b/src/box/applier.cc
> index 7e70211b7..9a9ec1dac 100644
> --- a/src/box/applier.cc
> +++ b/src/box/applier.cc
> @@ -806,7 +806,6 @@ applier_txn_commit_cb(struct trigger *trigger, void *event)
> (void) trigger;
> struct txn *txn = (struct txn *)event;
> assert(txn->fiber != NULL);
> - assert(strncmp(txn->fiber->name, "applierw", 8) == 0);
> /*
> * Let the txn module free the transaction object. It is
> * not needed for anything else.
--
Serge Petrenko
More information about the Tarantool-patches
mailing list