tarantool / tarantool / 9993828141

87%

master: 88%

Build Type

push

github

Committed by locker

Commit Message

vinyl: wake up waiters after clearing checkpoint_in_progress flag

The function `vy_space_build_index`, which builds a new index on DDL,
calls `vy_scheduler_dump` on completion. If there's a checkpoint in
progress, the latter will wait on `vy_scheduler::dump_cond` until
`vy_scheduler::checkpoint_in_progress` is cleared. The problem is
`vy_scheduler_end_checkpoint` doesn't broadcast `dump_cond` when it
clears the flag. Usually, everything works fine because the condition
variable is broadcast on any dump completion, and vinyl checkpoint
implies a dump, but under certain conditions this may lead to a fiber
hang. Let's broadcast `dump_cond` in `vy_scheduler_end_checkpoint`
to be on the safe side.

While we are at it, let's also inject a dump delay to the original
test to make it more robust.

Closes #10267
Follow-up #10234

NO_DOC=bug fix

(cherry picked from commit fc3196dca)

Run Details

67465 of 120003 branches covered (56.22%)

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

27 existing lines in 12 files now uncovered.

100196 of 115071 relevant lines covered (87.07%)

2480256.06 hits per line