• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

tarantool / tarantool / 10770964139
88%

Build:
DEFAULT BRANCH: master
Ran 09 Sep 2024 10:19AM UTC
Jobs 1
Files 514
Run time 4min
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

09 Sep 2024 10:02AM UTC coverage: 87.231% (+0.007%) from 87.224%
10770964139

push

github

locker
vinyl: fix ERRINJ_VY_DELAY_PK_LOOKUP

Enabling `ERRINJ_VY_DELAY_PK_LOOKUP` makes Vinyl yield in a place where
it wouldn't normally do. If the transaction is aborted in the meantime,
we'll get the assertion failure:

```
./src/box/vy_point_lookup.c:219: vy_point_lookup: Assertion 'tx == NULL || tx->state == VINYL_TX_READY' failed.
```

To prevent this from happening, let's replace this invalid error
injection with the new one `ERRINJ_VY_POINT_LOOKUP_DELAY` that injects
a delay to `vy_point_lookup()` before reading disk. This doesn't have
exactly the same effect as the old error injection because it also
delays direct lookups in the primary index. Fortunately, the old error
injection is used in the only test, where the new one works as expected
if we make the secondary index created in the test non-unique and enable
deferred writes (this makes the `s:replace{2, 2}` statement bypass
a lookup in the primary index).

Also, let's replace `VY_POINT_ITER_WAIT` with the new error injection
because they have very a similar meaning and `VY_POINT_LOOKUP_DELAY`
works in the test using it with a very small adjustment (we need to
clear it explicitly after `box.snapshot()`).

Closes #10517

NO_DOC=errinj fix
NO_CHANGELOG=errinj fix

68671 of 122132 branches covered (56.23%)

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

37 existing lines in 12 files now uncovered.

101448 of 116298 relevant lines covered (87.23%)

2476058.01 hits per line

Jobs
ID Job ID Ran Files Coverage
1 10770964139.1 09 Sep 2024 10:19AM UTC 0
87.23
GitHub Action Run
Source Files on build 10770964139
Detailed source file information is not available for this build.
  • Back to Repo
  • 92619635 on github
  • Prev Build on master (#10768952750)
  • Next Build on master (#10770966886)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc