Skip to content

NDRS-164: fetcher optimization#135

Merged
EdHastingsCasperAssociation merged 1 commit intocasper-network:masterfrom
EdHastingsCasperAssociation:ndrs-164-fetcher-optimization
Aug 3, 2020
Merged

NDRS-164: fetcher optimization#135
EdHastingsCasperAssociation merged 1 commit intocasper-network:masterfrom
EdHastingsCasperAssociation:ndrs-164-fetcher-optimization

Conversation

@EdHastingsCasperAssociation
Copy link
Collaborator

This PR optimizes the fetcher component to signal all responders with the item as soon as it becomes available from any source.

https://casperlabs.atlassian.net/browse/NDRS-164

@EdHastingsCasperAssociation EdHastingsCasperAssociation merged commit fb8f461 into casper-network:master Aug 3, 2020
@EdHastingsCasperAssociation EdHastingsCasperAssociation deleted the ndrs-164-fetcher-optimization branch August 3, 2020 08:35
for responder in responders {
effects.extend(responder.respond(None).ignore());
}
}
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we remove the entry id at this point, if it's empty now?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good catch. I fixed this in #137.

sacherjj pushed a commit that referenced this pull request Feb 8, 2024
Fix allocation in `FromBytes for Vec<T>`
rafal-ch pushed a commit that referenced this pull request Sep 11, 2024
rafal-ch pushed a commit that referenced this pull request Sep 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants