tarantool
/
tarantool
/
23217133177
88%
master: 88%
master: 88%
|
Ran
|
Jobs
1
|
Files
520
|
Run time
1min
|
Badge
Embed ▾
README BADGES
|
coverage: 87.605% (-0.02%) from 87.624%
push
github
Gerold103election: drop promotion flag before entering Raft box_promote() used to keep is_in_box_promote flag while running box_raft_try_promote(). Sometimes this led to the Raft worker being too fast, entering box_promote_qsync(), and failing on is_in_box_promote being set. Even though it is literally a part of the same promote. Lets drop the flag right before entering box_raft_try_promote(). Both before and after the behaviour is really the same. Just previously the Raft worker had to retry the limbo promotion. The difference is only detectable in the logs. This is why there is also no special test for this. The existing ones cover the guards completely fine. Closes #12333 NO_DOC=bugfix NO_CHANGELOG=too minor and no behaviour changes (cherry picked from commit 0cae9f838)
68809 of 116369 branches covered (59.13%)
2 of 2 new or added lines in 1 file covered. (100.0%)
311 existing lines in 14 files now uncovered.103525 of 118172 relevant lines covered (87.61%)
1655845.19 hits per line
Uncovered Existing Lines
| Lines | Coverage | ∆ | File |
|---|---|---|---|
| 1 |
90.63 |
-0.63% | src/box/engine.c |
| 1 |
94.74 |
-1.75% | src/box/engine.h |
| 1 |
90.14 |
-0.08% | src/box/vy_log.c |
| 1 |
87.5 |
-1.59% | src/box/xlog.h |
| 1 |
96.1 |
-0.05% | src/lib/salad/bps_tree.h |
| 2 |
94.66 |
-0.59% | src/box/raft.c |
| 2 |
92.52 |
-0.05% | src/box/vinyl.c |
| 2 |
87.5 |
0.07% | src/box/vy_run.c |
| 2 |
85.61 |
-0.72% | src/lib/core/coio.c |
| 2 |
85.81 |
-0.32% | src/lib/core/say.c |
| 3 |
91.27 |
-0.31% | src/box/vy_scheduler.c |
| 9 |
88.14 |
-7.63% | src/box/vy_regulator.c |
| 15 |
94.58 |
0.07% | src/box/txn_limbo.c |
| 269 |
90.82 |
-0.53% | src/box/box.cc |
Jobs
| ID | Job ID | Ran | Files | Coverage | |
|---|---|---|---|---|---|
| 1 | 23217133177.1 | 520 |
87.61 |
GitHub Action Run |
