Skip to content

feat: race a http gateway with p2p in iroh-one's content loader#338

Merged
Arqu merged 1 commit into
n0-computer:mainfrom
capyloon:iroh-one-racing-gateway
Oct 15, 2022
Merged

feat: race a http gateway with p2p in iroh-one's content loader#338
Arqu merged 1 commit into
n0-computer:mainfrom
capyloon:iroh-one-racing-gateway

Conversation

@fabricedesre

Copy link
Copy Markdown
Contributor

This adds to iroh-one a new optional configuration item: resolver_gateway
If resolver_gateway is set, the resolver will race the p2p node with fetching a raw block from the gateway.

A new source is also introduced to classify blocks coming from a gateway out from those fetched from p2p or store. Hits from the gateway are considered as not cached for telemetry purposes.

ping @b5

@Arqu Arqu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good, can we get the rustfmt in order and I can merge then.

@fabricedesre
fabricedesre force-pushed the iroh-one-racing-gateway branch from c050acd to 72ff3c7 Compare October 14, 2022 23:14
@fabricedesre

Copy link
Copy Markdown
Contributor Author

Looks good, can we get the rustfmt in order and I can merge then.

fixed in latest push.

This adds to iroh-one a new optional configuration item: resolver_gateway
If resolver_gateway is set, the resolver will race the p2p node with
fetching a raw block from the gateway.

A new source is also introduced to classify blocks coming from a gateway out
from those fetched from p2p or store. Hits from the gateway are considered as
not cached for telemetry purposes.
@fabricedesre
fabricedesre force-pushed the iroh-one-racing-gateway branch from 72ff3c7 to 5416108 Compare October 14, 2022 23:43
@Arqu
Arqu merged commit 40cb232 into n0-computer:main Oct 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants