tarantool / tarantool / 14487687066

87%

master: 88%

Build Type

push

github

Committed by sergepetrenko

Commit Message

box: set TXN_WAIT_SYNC with xrow rather than limbo state during recovery

During recovery we set the `TXN_WAIT_SYNC` flag based on the limbo's state
rather than respecting the xrow's flags. This does not play well with fully
local transactions that were committed using the `TXN_FORCE_ASYNC` flag:
during runtime they bypass the limbo as expected, but during recovery we do
not set this flag, and they are appended to the limbo, if it is not empty,
even though they are not supposed to be, since their xrows does not have
the `wait_sync` flag.

To fix this, during recovery let's set all the required flags related to
synchronous replication while processing the xrow, and ignore the limbo
emptiness, if we are in recovery and processing local transactions (i.e.,
not applying remote transactions).

Closes #11053

NO_CHANGELOG=<this version is not affect by backported bugfix>
NO_DOC=<this version is not affect by backported bugfix>
NO_TEST=<this version is not affect by backported bugfix>

(cherry picked from commit 4ba187909)

Run Details

68936 of 122197 branches covered (56.41%)

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

67 existing lines in 20 files now uncovered.

101624 of 116249 relevant lines covered (87.42%)

1969475.85 hits per line