tarantool / crud / 20987168712

88%

master: 88%

Build Type

Pull #475

github

Committed by Satbek

Commit Message

safe/fast mode: remove fiber kill

In fast mode, every fiber was named “fast” so it could be
killed when safe mode was enabled. However, crud-storage
methods do not yield until a space operation is performed.

The rebalancing trigger is executed on insert/replace into
the _bucket space; this operation does yield and switches
the implementation of the bucket_ref/bucket_unref functions.

Therefore, once a fiber starts in fast mode, it remains
in fast mode until the first box.space operation: a write
for memtx or a read/write for vinyl.

As a result, there is no need to mark and kill iproto fibers
that handle fast requests, because there is no situation in
which a fiber remains in fast mode while rebalancing is in progress.

Yield checks were added to the tests (yield_checks) to
ensure that no yields occur during request until box operations.

Pull Request Pull Request #475: safe/fast mode: remove fiber kill

Run Details

75 of 75 new or added lines in 14 files covered. (100.0%)

7 existing lines in 2 files now uncovered.

5215 of 5894 relevant lines covered (88.48%)

14042.57 hits per line