Skip to content

[backport 3.5] box: populate sysview iterator with position method#11958

Merged
sergepetrenko merged 1 commit intorelease/3.5from
backport/release/3.5/11954
Oct 22, 2025
Merged

[backport 3.5] box: populate sysview iterator with position method#11958
sergepetrenko merged 1 commit intorelease/3.5from
backport/release/3.5/11954

Conversation

@TarantoolBot
Copy link
Collaborator

@TarantoolBot TarantoolBot commented Oct 21, 2025

(This PR is a backport of #11954 to release/3.5 to a future 3.5.1 release.)


Currently Tarantool crashes when one tries to use pagination on a sysview space because it doesn't have iterator_position method and it's even not initialized to a generic version throwing an error. Anyway, sysview spaces support only tree indexes, so pagination definitely may work there - let's populate this engine with iterator_position method which simply call the same method of the underlying iterator.

Closes #11953

NO_DOC=bugfix

Currently Tarantool crashes when one tries to use pagination on a
`sysview` space because it doesn't have `iterator_position` method
and it's even not initialized to a generic version throwing an error.
Anyway, `sysview` spaces support only `tree` indexes, so pagination
definitely may work there - let's populate this engine with
`iterator_position` method which simply calls the same method of
the underlying iterator.

Closes #11953

NO_DOC=bugfix

(cherry picked from commit ffa157d)
@TarantoolBot TarantoolBot requested a review from a team as a code owner October 21, 2025 12:50
@TarantoolBot TarantoolBot changed the title [Backport release/3.5] box: populate sysview iterator with position method [backport 3.5] box: populate sysview iterator with position method Oct 21, 2025
@coveralls
Copy link

Coverage Status

coverage: 87.612% (-0.009%) from 87.621%
when pulling 874be5a on backport/release/3.5/11954
into 019f70d
on release/3.5
.

@sergepetrenko sergepetrenko merged commit 7c8f831 into release/3.5 Oct 22, 2025
25 checks passed
@sergepetrenko sergepetrenko deleted the backport/release/3.5/11954 branch October 22, 2025 09:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants