Skip to content
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.

Sign up for GitHub

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

Jump to bottom

[DNM]: Experimenting with TX submit loops #1369

Closed
wants to merge 21 commits into from
Closed

[DNM]: Experimenting with TX submit loops #1369

wants to merge 21 commits into from

Conversation

jsdw
Copy link
Collaborator

@jsdw jsdw commented Jan 16, 2024

No description provided.

lexnv and others added 21 commits January 15, 2024 19:29
@lexnv
backend(fix): Early unpin for pruned blocks with no active BlockRef
fc057b9 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
backend/tests: Check unpinning only after max_life time
bca0fe6 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
backend/tests: Remove unpinning when droped tests
8232d29 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
backend/tests: Ensure new blocks are not unpinned
2a5ac61 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
backend: Add only pruned blocks to unpin hashset and add a test
a74fb4b 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
backend: Introduce the UnpinPolicy
b4cc97e 
Signed-off-by: Alexandru Vasile <[email protected]>
@lexnv
Update subxt/src/backend/unstable/follow_stream_unpin.rs
b2b9395
@lexnv
backend: Fix clippy
43f850d 
Signed-off-by: Alexandru Vasile <[email protected]>
@jsdw
Only unpin blocks that have been pruned by the backend
867e80b
@jsdw
Fix comments
23a2f09
@jsdw
Mark initialized and finalized blocks as 'can_be_unpinned' too and tests
ee47111
@jsdw
fmt
c79e6ca
@jsdw
tweak a couple more comments
00df0b9
@jsdw
tidy tests and fix undeterministic check
9da0bb7
@jsdw
Fix wrong names in comments
b49fcfc
@jsdw
tweak another test to focus it a bit more
cb4beaa
@jsdw
clippy
29ad17d
@jsdw
wee rename of args in a couple of test helpers
6967ae4
@jsdw
add logging etc
5884de5
@jsdw
no-capture
e1c7532
@jsdw
run one test lots
62fd4c4
@jsdw jsdw closed this Jan 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jsdw @lexnv