tarantool / tarantool / 9417095440

87%

master: 88%

Build Type

push

github

Committed by locker

Commit Message

vinyl: fix crash on extending secondary key parts with primary

If a secondary index is altered in such a way that its key parts are
extended with the primary key parts, rebuild isn't required because
`cmp_def` doesn't change, see `vinyl_index_def_change_requires_rebuild`.
In this case `vinyl_index_update_def` will try to update `key_def` and
`cmp_def` in-place with `key_def_copy`. This will lead to a crash
because the number of parts in the new `key_def` is greater.

We can't use `key_def_dup` instead of `key_def_copy` there because
there may be read iterators using the old `key_def` by pointer so
there's no other option but to force rebuild in this case.

The bug was introduced in commit 64817066ff60 ("vinyl: use update_def
index method to update vy_lsm on ddl").

Closes #10095

NO_DOC=bug fix

(cherry picked from commit 9b817848c)

Run Details

67392 of 119889 branches covered (56.21%)

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

35 existing lines in 13 files now uncovered.

100099 of 114977 relevant lines covered (87.06%)

2429055.72 hits per line