tarantool / tarantool / 17731016174
88%
master: 88%

LAST BUILD BRANCH: gh-12728-replication-ignores-credentials-change
DEFAULT BRANCH: master
Ran
Jobs 1
Files 513
Run time 2min
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
coverage: 87.622% (+0.03%) from 87.597%
17731016174

push

github

locker 
box: fix old tuple format in on_replace trigger on space upgrade

Currently format of old tuple in `on_replace` trigger may be in new or old
format during space upgrade depending on whether replace is done before
or after upgrade cursor. Space upgrade is designed to be instant from
user POV: it returns upgraded tuples in `space.get()` for example even
if tuple is not upgraded internally. So to make the trigger consistent
with overall design let's always return old tuple in new format.

The old tuple visible in trigger is set by `memtx_space_replace_tuple()`
function. Instead of fixing it let's get rid of it for a few reasons:

- The interface is a bit odd. `old_tuple` is compressed and `new_tuple`
is not. Also reference for `new_tuple` is taken by the function and
is not for `old_tuple`.
- With this function we do some a bit heavy actions twice. For example
on update we decompress old tuple before update but then decompress it
again to set to `stmt->old_tuple` in this function.

Closes tarantool/tarantool-ee#1458

NO_TEST=the fix is tested by the test in EE repository
NO_CHANGELOG=the changelog will be added in EE repository
NO_DOC=bugfix

(cherry picked from commit 7a6319528)

71262 of 125041 branches covered (56.99%)

96 of 106 new or added lines in 2 files covered. (90.57%)

39 existing lines in 16 files now uncovered.

104735 of 119530 relevant lines covered (87.62%)

1371286.09 hits per line

Uncovered Changes

Lines Coverage ∆ File
10
94.09
 -0.16% src/box/memtx_space.c

Coverage Regressions

Lines Coverage ∆ File
6
94.09
 -0.16% src/box/memtx_space.c
6
93.05
 -1.16% src/box/xrow_update_field.c
3
93.62
 -1.6% src/box/vy_range.c
3
90.9
 -0.31% src/box/vy_scheduler.c
3
92.83
 -0.29% src/lib/core/fiber.c
3
22.64
 -0.06% src/lib/uri/uri_parser.c
2
91.33
 0.49% src/box/box.cc
2
92.54
 -0.04% src/box/vinyl.c
2
84.8
 -0.1% src/box/xlog.c
2
85.15
 -0.27% src/lib/core/say.c
2
85.71
 -9.52% src/lib/core/tt_sigaction.c
1
95.72
 -0.12% src/box/replication.cc
1
96.3
 -3.7% src/cpu_feature.c
1
80.0
 -2.05% src/lib/core/clock.c
1
97.36
 0.3% src/lib/msgpuck/msgpuck.h
1
97.41
 -0.86% src/lib/vclock/vclock.c
Jobs
ID Job ID Ran Files Coverage
1 17731016174.1 513
87.62
 GitHub Action Run
Source Files on build 17731016174