[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