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.

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

Activate crawler task based on a list of recent syncer obtain/extend lengths #2603

Closed
Tracked by #2309
mpguerra opened this issue Aug 11, 2021 · 2 comments
Closed
Tracked by #2309
Assignees
Labels
C-enhancement Category: This is an improvement

Comments

@mpguerra
Copy link
Contributor

mpguerra commented Aug 11, 2021

Motivation

We want to control when we activate the network crawler task that will help us to populate the mempool. We want to be able to do this both for testing purposes as well as when zebra reaches the chain tip.

We also want to stop the mempool crawler if the syncer starts syncing large numbers of blocks again.

The crawler will receive a latest_sync_heights: watch::Receiver<Vec<usize>> to use to determine if it should be enabled or not, and the crawler should stop if that endpoint is closed.

The crawler will also receive a sender channel for transaction ID responses from peers. It should also stop if that channel is closed.

Designs

Mempool Design diagram

Related Work

#2592

@mpguerra mpguerra mentioned this issue Aug 11, 2021
59 tasks
@mpguerra mpguerra added C-enhancement Category: This is an improvement S-needs-triage Status: A bug report needs triage labels Aug 13, 2021
@mpguerra mpguerra added this to the 2021 Sprint 17 milestone Aug 16, 2021
@jvff
Copy link
Contributor

jvff commented Aug 24, 2021

The crawler will receive a latest_sync_heights: watch::Receiver<Vec<usize>> to use to determine if it should be enabled or not, and the crawler should stop if that endpoint is closed.

@jvff
Copy link
Contributor

jvff commented Aug 31, 2021

Closed by #2667.

@jvff jvff closed this as completed Aug 31, 2021
@mpguerra mpguerra removed the S-needs-triage Status: A bug report needs triage label Sep 7, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: This is an improvement
Projects
None yet
Development

No branches or pull requests

2 participants