Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Update RequestResponseConfig interface to match substrate #5849

Merged
paritytech-processbot merged 6 commits into from
Aug 5, 2022
Merged

Update RequestResponseConfig interface to match substrate #5849

Changes from 3 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
2edcd1f
Update RequestResponseConfig interface
dmitry-markin Aug 1, 2022
59ea5be
minor: Bump CI
dmitry-markin Aug 1, 2022
34564f8
Merge branch 'master' into dm-substrate-protocol-names
dmitry-markin Aug 2, 2022
4836ded
Merge remote-tracking branch 'origin/master' into dm-substrate-protoc…
Aug 5, 2022
7f88793
update lockfile for {"substrate"}
Aug 5, 2022
531bf03
update lockfile for {"substrate"}
Aug 5, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 6 additions & 0 deletions node/network/protocol/src/request_response/mod.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -132,6 +132,7 @@ impl Protocol {
let cfg = match self {
Protocol::ChunkFetchingV1 => RequestResponseConfig {
name: p_name,
fallback_names: Vec::new(),
max_request_size: 1_000,
max_response_size: POV_RESPONSE_SIZE as u64 * 3,
// We are connected to all validators:
Expand All @@ -140,6 +141,7 @@ impl Protocol {
},
Protocol::CollationFetchingV1 => RequestResponseConfig {
name: p_name,
fallback_names: Vec::new(),
max_request_size: 1_000,
max_response_size: POV_RESPONSE_SIZE,
// Taken from initial implementation in collator protocol:
Expand All @@ -148,13 +150,15 @@ impl Protocol {
},
Protocol::PoVFetchingV1 => RequestResponseConfig {
name: p_name,
fallback_names: Vec::new(),
max_request_size: 1_000,
max_response_size: POV_RESPONSE_SIZE,
request_timeout: POV_REQUEST_TIMEOUT_CONNECTED,
inbound_queue: Some(tx),
},
Protocol::AvailableDataFetchingV1 => RequestResponseConfig {
name: p_name,
fallback_names: Vec::new(),
max_request_size: 1_000,
// Available data size is dominated by the PoV size.
max_response_size: POV_RESPONSE_SIZE,
Expand All @@ -163,6 +167,7 @@ impl Protocol {
},
Protocol::StatementFetchingV1 => RequestResponseConfig {
name: p_name,
fallback_names: Vec::new(),
max_request_size: 1_000,
// Available data size is dominated code size.
max_response_size: STATEMENT_RESPONSE_SIZE,
Expand All @@ -180,6 +185,7 @@ impl Protocol {
},
Protocol::DisputeSendingV1 => RequestResponseConfig {
name: p_name,
fallback_names: Vec::new(),
max_request_size: 1_000,
/// Responses are just confirmation, in essence not even a bit. So 100 seems
/// plenty.
Expand Down