tarantool / tarantool / 24881377735

88%

master: 88%

Build Type

push

github

Committed by sergepetrenko

Commit Message

txn: forbid waiting for acks on fully local transactions

We recommend making all spaces synchronous (including the system ones)
whenever the user has at least one of them. However, once `_truncate`
becomes synchronous, it is impossible to truncate any local space,
either on the master or on a replica. It fails with the error: `An
attempt to commit a fully local transaction synchronously`.

The `_truncate` space itself is unusual: it marks rows related to local
spaces as GROUP_LOCAL (they are not replicated), while all other rows
remain GROUP_GLOBAL. As a result, when we truncate a local space all rows
in the transaction are local, so the error is thrown.

We should not mark such transactions as synchronous. Instead, it should
follow the asynchronous transaction path: add them to limbo iff it's not
empty. Let's filter such case.

The commit also fixes the incorrect comment below, which says that we
don't check for fully local transaction, when we clearly do.

Closes #12585

NO_DOC=bugfix

Co-authored-by: Serge Petrenko <sergepetrenko@tarantool.org>
(cherry picked from commit 6adf3f407)

Coverage Stats

70775 of 119056 branches covered (59.45%)

6 of 6 new or added lines in 1 file covered. (100.0%)

31 existing lines in 15 files now uncovered.

106394 of 121343 relevant lines covered (87.68%)

1381550.38 hits per line