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

Get recent obtain/extend lengths out of the sync service #2595

Closed
Tracked by #2309
teor2345 opened this issue Aug 9, 2021 · 0 comments · Fixed by #2602
Closed
Tracked by #2309

Get recent obtain/extend lengths out of the sync service #2595

teor2345 opened this issue Aug 9, 2021 · 0 comments · Fixed by #2602
Assignees
@teor2345
Labels
A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement
Milestone
2021 Sprint 16

Comments

@teor2345
Copy link
Contributor

teor2345 commented Aug 9, 2021

Motivation

To activate the mempool, we need to know the most recent obtain/extend lengths that the sync service has got from peers.

Designs

https://jamboard.google.com/d/1HUhdqx8afJUqgiBdBcjCw0aAfDod9R9KBJeAH24U1Qg/viewer?f=0

https://docs.google.com/document/d/10PP6wKlnS5gannRph_EQ8V9diR7byddf3W2YFZ564FU/edit#heading=h.1v3bcppsa6wq

Related Work

This change is required for ticket #2592.
@teor2345 teor2345 added A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement S-needs-triage Status: A bug report needs triage P-Medium labels Aug 9, 2021
@teor2345 teor2345 added this to the 2021 Sprint 16 milestone Aug 9, 2021
@teor2345 teor2345 self-assigned this Aug 9, 2021
This was referenced Aug 9, 2021
Closed
Merged
@mpguerra mpguerra mentioned this issue Aug 11, 2021
Closed
59 tasks
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Aug 16, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@teor2345 teor2345

Labels
A-rust Area: Updates to Rust code C-enhancement Category: This is an improvement
Projects
None yet
Milestone
2021 Sprint 16
Development

Successfully merging a pull request may close this issue.

Provide recent syncer response lengths as a watch channel ZcashFoundation/zebra
2 participants
@mpguerra @teor2345