rpcdaemon: page_token management in Data API Range ops #2477

Sixtysixter · 2024-11-04T23:04:30Z

Fix page_token management in Range queries: add page_token parameter to PaginatedSequence::Paginator and PaginatedSequence::PageResult.

silkworm/db/kv/api/endpoint/paginated_sequence.hpp

silkworm/db/kv/api/endpoint/paginated_sequence_test.cpp

Sixtysixter and others added 4 commits November 5, 2024 00:02

Fix for page_token management in RemoteTransaction::XXX_range methods

a81c612

Moved next token page in PaginatedSequence

d6f6ceb

make fmt

a6280f3

Fix in test

3e7665f

Sixtysixter requested a review from canepat November 5, 2024 08:29

Sixtysixter marked this pull request as ready for review November 5, 2024 08:29

canepat added erigon3 Erigon3 feature maintenance Some maintenance work (fix, refactor, rename, test...) labels Nov 5, 2024

canepat requested changes Nov 5, 2024

View reviewed changes

Sixtysixter added 6 commits November 5, 2024 15:22

Fix after review

050fc0d

Removed useless functions

e03957b

Fix in paginator signature

d206196

Fix in paginator signature

a30d04a

Added type for page token in paginator signature

9d0137f

Fix

400636d

canepat changed the title ~~rpcdaemon: page_token management in range ops~~ rpcdaemon: page_token management in Data API Range ops Nov 7, 2024

canepat self-requested a review November 7, 2024 07:40

canepat approved these changes Nov 7, 2024

View reviewed changes

canepat merged commit 91bfe19 into master Nov 7, 2024
5 checks passed

canepat deleted the fix/page_token_in_range_ops branch November 7, 2024 07:41

Provide feedback