Skip to content

Commit e5e0a6d

Browse files
authored
enable approval-voting-parallel on polkadot (#7504)
approval-voting-parallel has been running on all network types except polkadot since release `1.17.0`, so it has around 2 months of baking and running without any reported issues, so let's enable it on polkadot as well. After running in polkadot for awhile the flag will be entirely removed. --------- Signed-off-by: Alexandru Gheorghe <[email protected]>
1 parent 1833515 commit e5e0a6d

File tree

3 files changed

+15
-11
lines changed

3 files changed

+15
-11
lines changed

polkadot/cli/src/cli.rs

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@
1818
1919
pub use polkadot_node_primitives::NODE_VERSION;
2020

21-
use clap::Parser;
21+
use clap::{ArgAction, Parser};
2222
use std::path::PathBuf;
2323

2424
#[allow(missing_docs)]
@@ -148,9 +148,9 @@ pub struct RunCmd {
148148

149149
/// Enable approval-voting message processing in parallel.
150150
///
151-
///**Dangerous!** This is an experimental feature and should not be used in production, unless
152-
/// explicitly advised to.
153-
#[arg(long)]
151+
/// This is a flag used for gradually enabling approval-voting-parallel in production,
152+
/// should not be used unless explicitly advised to. It will be removed in the future.
153+
#[arg(long, default_value = "true", action=ArgAction::Set)]
154154
pub enable_approval_voting_parallel: bool,
155155
}
156156

polkadot/node/service/src/lib.rs

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -759,13 +759,6 @@ pub fn new_full<
759759
Some(backoff)
760760
};
761761

762-
// Running approval voting in parallel is enabled by default on all networks except Polkadot
763-
// unless explicitly enabled by the commandline option.
764-
// This is meant to be temporary until we have enough confidence in the new system to enable it
765-
// by default on all networks.
766-
let enable_approval_voting_parallel =
767-
!config.chain_spec.is_polkadot() || enable_approval_voting_parallel;
768-
769762
let disable_grandpa = config.disable_grandpa;
770763
let name = config.network.node_name.clone();
771764

prdoc/pr_7504.prdoc

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,11 @@
1+
title: Enable approval-voting-parallel by default on polkadot
2+
3+
doc:
4+
- audience: Node Dev
5+
description: |
6+
Enable approval-voting-parallel by default on polkadot
7+
crates:
8+
- name: polkadot-service
9+
bump: patch
10+
- name: polkadot-cli
11+
bump: patch

0 commit comments

Comments
 (0)