tarantool / tarantool / 19855321294

88%

Build Type

push

github

Committed by sergepetrenko

Commit Message

memtx: unify `replace_all_keys` code paths w/ and w/o MVCC

Currently, we have 2 separate code paths for `memtx_replace_all_keys` w/
and w/o transaction management inside and outside the memtx transaction
manager.

Let's instead have a common replace code path in the transaction manager.
Without MVCC, we will only replace tuples in indexes, as before. With MVCC,
we will omit physical index replaces in case of deletes or perform deferred
conflict resolution, determine result tuple and track read and write sets.

This way the transaction management API will become easier to scale for
multikey indexes (gh-6385) which require a series of ordinary index
replaces.

To improve the coverage of memtx MVCC and make sure this refactoring does
not cause regressions, let's add test cases for the failure paths of index
replaces.

Closes #11732
Needed for #6385

NO_CHANGELOG=refactoring
NO_DOC=refactoring

Run Details

70230 of 118207 branches covered (59.41%)

44 of 47 new or added lines in 2 files covered. (93.62%)

64 existing lines in 17 files now uncovered.

105479 of 120365 relevant lines covered (87.63%)

1361544.58 hits per line