tarantool / tarantool / 9993841129

86%

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

62723 of 113974 branches covered (55.03%)

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

35 existing lines in 8 files now uncovered.

93632 of 109109 relevant lines covered (85.82%)

2602066.35 hits per line