Skip to content
Snippets Groups Projects
Unverified Commit e5e0a6d3 authored by Alexandru Gheorghe's avatar Alexandru Gheorghe Committed by GitHub
Browse files

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: default avatarAlexandru Gheorghe <alexandru.gheorghe@parity.io>
parent 18335159
No related merge requests found
Pipeline #514716 waiting for manual action with stages
in 46 minutes and 10 seconds
......@@ -18,7 +18,7 @@
pub use polkadot_node_primitives::NODE_VERSION;
use clap::Parser;
use clap::{ArgAction, Parser};
use std::path::PathBuf;
#[allow(missing_docs)]
......@@ -148,9 +148,9 @@ pub struct RunCmd {
/// Enable approval-voting message processing in parallel.
///
///**Dangerous!** This is an experimental feature and should not be used in production, unless
/// explicitly advised to.
#[arg(long)]
/// This is a flag used for gradually enabling approval-voting-parallel in production,
/// should not be used unless explicitly advised to. It will be removed in the future.
#[arg(long, default_value = "true", action=ArgAction::Set)]
pub enable_approval_voting_parallel: bool,
}
......
......@@ -759,13 +759,6 @@ pub fn new_full<
Some(backoff)
};
// Running approval voting in parallel is enabled by default on all networks except Polkadot
// unless explicitly enabled by the commandline option.
// This is meant to be temporary until we have enough confidence in the new system to enable it
// by default on all networks.
let enable_approval_voting_parallel =
!config.chain_spec.is_polkadot() || enable_approval_voting_parallel;
let disable_grandpa = config.disable_grandpa;
let name = config.network.node_name.clone();
......
title: Enable approval-voting-parallel by default on polkadot
doc:
- audience: Node Dev
description: |
Enable approval-voting-parallel by default on polkadot
crates:
- name: polkadot-service
bump: patch
- name: polkadot-cli
bump: patch
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment