tarantool / tarantool / 11952384890

86%

master: 87%

Build Type

push

github

Committed by locker

Commit Message

sql: do not use raw index for count

Currently, we use raw index for count operation instead of
`box_index_count`. As a result, we skip a check if current transaction
can continue and we don't begin transaction in engine if needed. So, if
count statement is the first in a transaction, it won't be tracked by
MVCC since it wasn't notified about the transaction. The commit fixes
the mistake. Also, the commit adds a check if count was successful and
covers it with a test.

In order to backport the commit to 2.11, space name was wrapped with
quotes since it is in lower case and addressing such spaces with SQL
without quotes is Tarantool 3.0 feature. Another unsupported feature is
prohibition of data access in transactional triggers - it was used in a
test case so it was rewritten.

Closes #10825

NO_DOC=bugfix

(cherry picked from commit 0656a9231)

Run Details

63154 of 114381 branches covered (55.21%)

5 of 5 new or added lines in 2 files covered. (100.0%)

37 existing lines in 14 files now uncovered.

94013 of 109370 relevant lines covered (85.96%)

2679225.08 hits per line