Skip to content

db: streams hiding pagination in kv::api::Transaction range Data API #2864

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 4 commits into from
Apr 29, 2025

Conversation

canepat
Copy link
Member

@canepat canepat commented Apr 21, 2025

db: implement direct kv::api::LocalTransaction using ranges/views
db: move paginated implementation to kv::api::RemoteTransaction
db: support skipping empty values in range queries for direct implementation
db: fix empty end key in range queries
db: move ranges' vector from iterator to view in MergeManyView
rpc: fix block parameter lifecycle in parity_listStorageKeys
rpc: rename PaginatedStream as Stream

canepat added 3 commits April 22, 2025 00:55
db: implement direct kv::api::LocalTransaction using ranges/views
db: move paginated implementation to kv::api::RemoteTransaction
db: support skipping empty values in range queries for direct implementation
db: fix empty end key in range queries
db: move ranges' vector from iterator to view in MergeManyView
rpc: fix block parameter lifecycle in parity_listStorageKeys
rpc: rename PaginatedStream as Stream
@canepat canepat added performance Performance issue or improvement erigon3 Erigon3 feature labels Apr 22, 2025
@canepat canepat marked this pull request as ready for review April 29, 2025 14:14
@battlmonstr battlmonstr self-requested a review April 29, 2025 14:14
@canepat canepat merged commit ae0f69c into master Apr 29, 2025
5 checks passed
@canepat canepat deleted the db_datastore_direct_range_as_of branch April 29, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
erigon3 Erigon3 feature performance Performance issue or improvement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants