Update for using Mandatory inherents (#967)

* Update for using Mandatory inherents.

* use
This commit is contained in:
Gavin Wood
2020-04-05 14:27:58 +02:00
committed by GitHub
parent 15a83079ba
commit 7f838b0c35
3 changed files with 12 additions and 3 deletions
+1 -1
View File
@@ -131,7 +131,7 @@ decl_module! {
type Error = Error<T>;
/// Provide candidate receipts for parachains, in ascending order by id.
#[weight = frame_support::weights::SimpleDispatchInfo::default()]
#[weight = frame_support::weights::SimpleDispatchInfo::FixedMandatory(10_000)]
fn more_attestations(origin, _more: MoreAttestations) -> DispatchResult {
ensure_none(origin)?;
ensure!(!DidUpdate::exists(), Error::<T>::TooManyAttestations);
+1 -1
View File
@@ -399,7 +399,7 @@ decl_module! {
type Error = Error<T>;
/// Provide candidate receipts for parachains, in ascending order by id.
#[weight = SimpleDispatchInfo::FixedNormal(1_000_000)]
#[weight = SimpleDispatchInfo::FixedMandatory(1_000_000)]
pub fn set_heads(origin, heads: Vec<AttestedCandidate>) -> DispatchResult {
ensure_none(origin)?;
ensure!(!<DidUpdate>::exists(), Error::<T>::TooManyHeadUpdates);
+10 -1
View File
@@ -277,7 +277,16 @@ impl<Client, TxPool, Backend> CreateProposalData<Client, TxPool, Backend> where
{
let inherents = runtime_api.inherent_extrinsics(&self.parent_id, inherent_data)?;
for inherent in inherents {
block_builder.push(inherent)?;
match block_builder.push(inherent) {
Err(sp_blockchain::Error::ApplyExtrinsicFailed(sp_blockchain::ApplyExtrinsicFailed::Validity(e)))
if e.exhausted_resources() => {
warn!("⚠️ Dropping non-mandatory inherent from overweight block.");
}
Err(e) => {
warn!("❗️ Inherent extrinsic returned unexpected error: {}. Dropping.", e);
}
Ok(_) => {}
}
}
let mut unqueue_invalid = Vec::new();