Skip to content
Snippets Groups Projects
Commit 9592b55f authored by Svyatoslav Nikolsky's avatar Svyatoslav Nikolsky Committed by Bastian Köcher
Browse files

fix parameter to match used value GRANDPA benchmarks (#1616)

parent 4f796ce8
No related merge requests found
...@@ -417,6 +417,12 @@ parameter_types! { ...@@ -417,6 +417,12 @@ parameter_types! {
/// ///
/// Note: This is lower than regular value, to speed up benchmarking setup. /// Note: This is lower than regular value, to speed up benchmarking setup.
pub const HeadersToKeep: u32 = 1024; pub const HeadersToKeep: u32 = 1024;
/// Maximal number of authorities at Rialto.
///
/// In benchmarks we're using sets of up to `1024` authorities to prepare for possible
/// upgrades in the future and see if performance degrades when number of authorities
/// grow.
pub const MaxAuthoritiesAtRialto: u32 = pallet_bridge_grandpa::benchmarking::MAX_VALIDATOR_SET_SIZE;
} }
#[cfg(not(feature = "runtime-benchmarks"))] #[cfg(not(feature = "runtime-benchmarks"))]
...@@ -426,11 +432,11 @@ parameter_types! { ...@@ -426,11 +432,11 @@ parameter_types! {
/// Assuming the worst case of every header being finalized, we will keep headers at least for a /// Assuming the worst case of every header being finalized, we will keep headers at least for a
/// week. /// week.
pub const HeadersToKeep: u32 = 7 * bp_rialto::DAYS; pub const HeadersToKeep: u32 = 7 * bp_rialto::DAYS;
/// Maximal number of authorities at Rialto.
pub const MaxAuthoritiesAtRialto: u32 = bp_rialto::MAX_AUTHORITIES_COUNT;
} }
parameter_types! { parameter_types! {
/// Maximal number of authorities at Rialto.
pub const MaxAuthoritiesAtRialto: u32 = bp_rialto::MAX_AUTHORITIES_COUNT;
/// Maximal size of SCALE-encoded Rialto header. /// Maximal size of SCALE-encoded Rialto header.
pub const MaxRialtoHeaderSize: u32 = bp_rialto::MAX_HEADER_SIZE; pub const MaxRialtoHeaderSize: u32 = bp_rialto::MAX_HEADER_SIZE;
......
...@@ -61,7 +61,7 @@ const MAX_VOTE_ANCESTRIES: u32 = 1000; ...@@ -61,7 +61,7 @@ const MAX_VOTE_ANCESTRIES: u32 = 1000;
// The maximum number of pre-commits to include in a justification. In practice this scales with the // The maximum number of pre-commits to include in a justification. In practice this scales with the
// number of validators. // number of validators.
const MAX_VALIDATOR_SET_SIZE: u32 = 1024; pub const MAX_VALIDATOR_SET_SIZE: u32 = 1024;
// `1..MAX_VALIDATOR_SET_SIZE` and `1..MAX_VOTE_ANCESTRIES` are too large && benchmarks are // `1..MAX_VALIDATOR_SET_SIZE` and `1..MAX_VOTE_ANCESTRIES` are too large && benchmarks are
// running for almost 40m (steps=50, repeat=20) on a decent laptop, which is too much. Since // running for almost 40m (steps=50, repeat=20) on a decent laptop, which is too much. Since
......
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