More compile

Author: bkontur

node/src/fake_runtime_api.rs

@@ -210,6 +210,12 @@ sp_api::impl_runtime_apis! {
 		}
 	}
 
+	impl sp_transaction_storage_proof::runtime_api::TransactionStorageApi<Block> for Runtime {
+		fn retention_period() -> sp_runtime::traits::NumberFor<Block> {
+			unimplemented!()
+		}
+	}
+
 	#[cfg(feature = "try-runtime")]
 	impl frame_try_runtime::TryRuntime<Block> for Runtime {
 		fn on_runtime_upgrade(_: frame_try_runtime::UpgradeCheckSelect) -> (Weight, Weight) {

node/src/service.rs

@@ -7,7 +7,9 @@ use sc_consensus_grandpa::SharedVoterState;
 use sc_service::{error::Error as ServiceError, Configuration, TaskManager};
 use sc_telemetry::{Telemetry, TelemetryWorker};
 use sc_transaction_pool_api::OffchainTransactionPoolFactory;
+use sp_api::ProvideRuntimeApi;
 use sp_consensus_babe::inherents::BabeCreateInherentDataProviders;
+use sp_transaction_storage_proof::runtime_api::TransactionStorageApi;
 use std::{sync::Arc, time::Duration};
 
 pub(crate) type FullClient = sc_service::TFullClient<
@@ -310,6 +312,7 @@ pub fn new_full<
 						sp_transaction_storage_proof::registration::new_data_provider(
 							&*client_clone,
 							&parent,
+							client_clone.runtime_api().retention_period(parent)?,
 						)?;
 
 					Ok((slot, timestamp, storage_proof))

pallets/transaction-storage/src/lib.rs

@@ -54,6 +54,9 @@ pub use weights::WeightInfo;
 
 const LOG_TARGET: &str = "runtime::transaction-storage";
 
+/// Default retention period for data (in blocks).
+pub const DEFAULT_RETENTION_PERIOD: u32 = 100800;
+
 /// Maximum bytes that can be stored in one transaction.
 // TODO: find out what is "allocator" and "allocator limit"
 // Setting higher limit also requires raising the allocator limit.

runtime/src/lib.rs

@@ -193,8 +193,8 @@ parameter_types! {
 		EquivocationReportPeriodInEpochs::get() * (EPOCH_DURATION_IN_BLOCKS as u64);
 
 
-	// This currently _must_ be set to DEFAULT_STORAGE_PERIOD
-	pub const StoragePeriod: BlockNumber = sp_transaction_storage_proof::DEFAULT_STORAGE_PERIOD;
+	// This currently _must_ be set to DEFAULT_RETENTION_PERIOD
+	pub const StoragePeriod: BlockNumber = pallet_transaction_storage::DEFAULT_RETENTION_PERIOD;
 	pub const AuthorizationPeriod: BlockNumber = 7 * DAYS;
 	pub const StoreRenewPriority: TransactionPriority = RemoveExpiredAuthorizationPriority::get() - 1;
 	pub const StoreRenewLongevity: TransactionLongevity = DAYS as TransactionLongevity;

runtimes/bulletin-polkadot/src/lib.rs

@@ -194,8 +194,8 @@ parameter_types! {
 		EquivocationReportPeriodInEpochs::get() * (EPOCH_DURATION_IN_BLOCKS as u64);
 
 
-	// This currently _must_ be set to DEFAULT_STORAGE_PERIOD
-	pub const StoragePeriod: BlockNumber = sp_transaction_storage_proof::DEFAULT_STORAGE_PERIOD;
+	// This currently _must_ be set to DEFAULT_RETENTION_PERIOD
+	pub const StoragePeriod: BlockNumber = pallet_transaction_storage::DEFAULT_RETENTION_PERIOD;
 	pub const AuthorizationPeriod: BlockNumber = 7 * DAYS;
 	pub const StoreRenewPriority: TransactionPriority = RemoveExpiredAuthorizationPriority::get() - 1;
 	pub const StoreRenewLongevity: TransactionLongevity = DAYS as TransactionLongevity;

runtimes/bulletin-westend/src/storage.rs

@@ -27,8 +27,8 @@ use sp_runtime::transaction_validity::{TransactionLongevity, TransactionPriority
 use testnet_parachains_constants::westend::locations::PeopleLocation;
 
 parameter_types! {
-	// This currently _must_ be set to DEFAULT_STORAGE_PERIOD
-	pub const StoragePeriod: crate::BlockNumber = sp_transaction_storage_proof::DEFAULT_STORAGE_PERIOD;
+	// This currently _must_ be set to DEFAULT_RETENTION_PERIOD
+	pub const StoragePeriod: crate::BlockNumber = pallet_transaction_storage::DEFAULT_RETENTION_PERIOD;
 	pub const AuthorizationPeriod: crate::BlockNumber = 7 * crate::DAYS;
 	// Priorities and longevities used by the transaction storage pallet extrinsics.
 	pub const SudoPriority: TransactionPriority = TransactionPriority::MAX;