From 0f8b6f2f6ef5d3011e135223dc459c02efcea37c Mon Sep 17 00:00:00 2001 From: Robert Klotzner Date: Wed, 24 Mar 2021 13:29:59 +0100 Subject: [PATCH] Bigger is better. (#2687) * Bigger is better. Made all request response sizes 10 times bigger. * The smaller the better. * Update comment. * Ah, bigger is still better. Max PoV size for rococo is around 50Meg, compression ratio is about 3.4. With 30 Meg we should be fine, even with crypto kitties in the PoV. --- .../network/protocol/src/request_response/mod.rs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/polkadot/node/network/protocol/src/request_response/mod.rs b/polkadot/node/network/protocol/src/request_response/mod.rs index 53d12926c4..70f0da0365 100644 --- a/polkadot/node/network/protocol/src/request_response/mod.rs +++ b/polkadot/node/network/protocol/src/request_response/mod.rs @@ -91,18 +91,18 @@ impl Protocol { let cfg = match self { Protocol::AvailabilityFetching => RequestResponseConfig { name: p_name, - max_request_size: 1_000, - max_response_size: 100_000, + max_request_size: 10_000, + max_response_size: 10_000_000, request_timeout: DEFAULT_REQUEST_TIMEOUT, inbound_queue: Some(tx), }, Protocol::CollationFetching => RequestResponseConfig { name: p_name, - max_request_size: 1_000, + max_request_size: 10_000, /// Collations are expected to be around 10Meg, probably much smaller with - /// compression. So 10Meg should be sufficient, we might be able to reduce this - /// further. - max_response_size: 10_000_000, + /// compression. So 30Meg should be well sufficient, we might be able to reduce + /// this further, if needed. + max_response_size: 30_000_000, // Taken from initial implementation in collator protocol: request_timeout: DEFAULT_REQUEST_TIMEOUT_CONNECTED, inbound_queue: Some(tx),