Skip to content

Commit c1b64eb

Browse files
[stable2409] Backport #5924 (#6356)
Backport #5924 into `stable2409` from AndreiEres. See the [documentation](https://github.com/paritytech/polkadot-sdk/blob/master/docs/BACKPORT.md) on how to use this bot. <!-- # To be used by other automation, do not modify: original-pr-number: #${pull_number} --> Co-authored-by: Andrei Eres <[email protected]>
1 parent 835e076 commit c1b64eb

File tree

2 files changed

+19
-4
lines changed

2 files changed

+19
-4
lines changed

polkadot/node/network/protocol/src/request_response/mod.rs

+6-4
Original file line numberDiff line numberDiff line change
@@ -123,10 +123,12 @@ const DEFAULT_REQUEST_TIMEOUT_CONNECTED: Duration = Duration::from_secs(1);
123123
/// Timeout for requesting availability chunks.
124124
pub const CHUNK_REQUEST_TIMEOUT: Duration = DEFAULT_REQUEST_TIMEOUT_CONNECTED;
125125

126-
/// This timeout is based on what seems sensible from a time budget perspective, considering 6
127-
/// second block time. This is going to be tough, if we have multiple forks and large PoVs, but we
128-
/// only have so much time.
129-
const POV_REQUEST_TIMEOUT_CONNECTED: Duration = Duration::from_millis(1200);
126+
/// This timeout is based on the following parameters, assuming we use asynchronous backing with no
127+
/// time budget within a relay block:
128+
/// - 500 Mbit/s networking speed
129+
/// - 10 MB PoV
130+
/// - 10 parallel executions
131+
const POV_REQUEST_TIMEOUT_CONNECTED: Duration = Duration::from_millis(2000);
130132

131133
/// We want timeout statement requests fast, so we don't waste time on slow nodes. Responders will
132134
/// try their best to either serve within that timeout or return an error immediately. (We need to

prdoc/pr_5924.prdoc

+13
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,13 @@
1+
# Schema: Polkadot SDK PRDoc Schema (prdoc) v1.0.0
2+
# See doc at https://raw.githubusercontent.com/paritytech/polkadot-sdk/master/prdoc/schema_user.json
3+
4+
title: Bump PoV request timeout
5+
6+
doc:
7+
- audience: Node Dev
8+
description: |
9+
With asynchronous backing and PoV size 10MB, we can increase the PoV request timeout from 1.2s to 2s.
10+
11+
crates:
12+
- name: polkadot-node-network-protocol
13+
bump: patch

0 commit comments

Comments
 (0)