Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Statement store #13701

Merged
merged 92 commits into from
May 4, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
92 commits
Select commit Hold shift + click to select a range
729e524
WIP Statement store
arkpar Mar 14, 2023
7591d22
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Mar 15, 2023
ff1edaa
Sync with networking changes in master
arkpar Mar 15, 2023
8407e40
WIP statement pallet
arkpar Mar 15, 2023
21c9aaf
Statement validation
arkpar Mar 16, 2023
c706097
pallet tests
arkpar Mar 16, 2023
e066abe
Validation queue
arkpar Mar 17, 2023
bfbbc5b
Store maintenance
arkpar Mar 20, 2023
d36a867
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Mar 21, 2023
d610e62
Basic statement refactoring + tests + docs
arkpar Mar 21, 2023
0b73266
Store metrics
arkpar Mar 22, 2023
66b9c98
Store tests
arkpar Mar 22, 2023
c8f0467
Store maintenance test
arkpar Mar 23, 2023
9bc6773
cargo fmt
arkpar Mar 23, 2023
8263aa4
Build fix
arkpar Mar 23, 2023
7779458
OCW Api
arkpar Mar 23, 2023
f01160e
Offchain worker
arkpar Mar 23, 2023
7431f55
Enable host functions
arkpar Mar 23, 2023
5cc24b2
fmt
arkpar Mar 23, 2023
dea2dea
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Mar 24, 2023
4febbea
Minor tweaks
arkpar Mar 24, 2023
b06e356
Fixed a warning
arkpar Mar 24, 2023
c6f2b53
Removed tracing
arkpar Mar 24, 2023
50b583d
Manual expiration
arkpar Mar 24, 2023
e141eb7
Reworked constraint management
arkpar Mar 28, 2023
c5555e9
Updated pallet constraint calculation
arkpar Mar 28, 2023
1171ba0
Added small test
arkpar Mar 28, 2023
399cbbd
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Mar 30, 2023
365e5e8
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 1, 2023
3d571ca
Added remove function to the APIs
arkpar Apr 1, 2023
85a31c9
Copy-paste spec into readme
arkpar Apr 1, 2023
b07775f
Comments
arkpar Apr 1, 2023
659b494
Made the store optional
arkpar Apr 2, 2023
25f3771
Removed network protocol controller
arkpar Apr 2, 2023
27e73cc
fmt
arkpar Apr 3, 2023
39f0145
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 3, 2023
51b16ff
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 3, 2023
0e84775
Clippy fixes
arkpar Apr 3, 2023
312a07e
fmt
arkpar Apr 3, 2023
9c21172
fmt
arkpar Apr 3, 2023
0d95b2e
More clippy fixes
arkpar Apr 3, 2023
61f6e5f
More clippy fixes
arkpar Apr 3, 2023
d93493a
More clippy fixes
arkpar Apr 3, 2023
f06ac25
Update client/statement-store/README.md
arkpar Apr 5, 2023
3c2373d
Apply suggestions from code review
arkpar Apr 5, 2023
a13e2c0
Removed sstore from node-template
arkpar Apr 5, 2023
ac499ab
Sort out data path
arkpar Apr 5, 2023
e562b65
Added offline check
arkpar Apr 5, 2023
06f3376
Removed dispatch_statement
arkpar Apr 5, 2023
167c4ad
Renamed into_generic
arkpar Apr 5, 2023
4c840c0
Fixed commit placement
arkpar Apr 5, 2023
03906c8
Use HashSet for tracking peers/statements
arkpar Apr 5, 2023
f33942c
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 5, 2023
206488d
fmt
arkpar Apr 5, 2023
2625783
Use ExtendedHostFunctions
arkpar Apr 5, 2023
a7f652a
Fixed benches
arkpar Apr 5, 2023
0bf748e
Tweaks
arkpar Apr 5, 2023
8c883a7
Apply suggestions from code review
arkpar Apr 17, 2023
c05e3d7
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 17, 2023
6c15ce9
Fixed priority mixup
arkpar Apr 17, 2023
f383ab6
Rename
arkpar Apr 17, 2023
a978b2a
newtypes for priorities
arkpar Apr 18, 2023
83f343f
Added MAX_TOPICS
arkpar Apr 18, 2023
94b10a7
Fixed key filtering logic
arkpar Apr 18, 2023
1c22fef
Remove empty entrie
arkpar Apr 18, 2023
f08810b
Removed prefix from signing
arkpar Apr 18, 2023
7849cc8
More documentation
arkpar Apr 18, 2023
64ac7f5
fmt
arkpar Apr 18, 2023
f05f5a3
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 19, 2023
21dd020
Moved store setup from sc-service to node
arkpar Apr 19, 2023
d72290c
Handle maintenance task in sc-statement-store
arkpar Apr 19, 2023
16e469e
Use statement iterator
arkpar Apr 19, 2023
8d18b5e
Renamed runtime API mod
arkpar Apr 19, 2023
9d3add2
fmt
arkpar Apr 19, 2023
cd7c9dc
Remove dump_encoded
arkpar Apr 19, 2023
d29ebc7
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 24, 2023
38d893d
fmt
arkpar Apr 24, 2023
fdb43c5
Apply suggestions from code review
arkpar Apr 24, 2023
74e4945
Apply suggestions from code review
arkpar Apr 24, 2023
4f1ac76
Fixed build after applying review suggestions
arkpar Apr 24, 2023
2f2d4f8
License exceptions
arkpar Apr 25, 2023
75ee1bf
fmt
arkpar Apr 25, 2023
b501198
Store options
arkpar Apr 25, 2023
692f58a
Moved pallet consts to config trait
arkpar Apr 25, 2023
704d53a
Removed global priority
arkpar Apr 25, 2023
71b39cf
Validate fields when decoding
arkpar Apr 25, 2023
a798b08
Limit validation channel size
arkpar Apr 25, 2023
7af3b5a
Made a comment into module doc
arkpar Apr 25, 2023
47a8e6b
Removed submit_encoded
arkpar Apr 25, 2023
4fb2aee
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 25, 2023
df2df2a
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar Apr 25, 2023
f81511d
Merge branch 'master' of github.com:paritytech/substrate into a-state…
arkpar May 3, 2023
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
88 changes: 88 additions & 0 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -46,6 +46,7 @@ members = [
"client/merkle-mountain-range/rpc",
"client/network",
"client/network/transactions",
"client/network/statement",
"client/network-gossip",
"client/network/bitswap",
"client/network/common",
Expand All @@ -63,6 +64,7 @@ members = [
"client/service",
"client/service/test",
"client/state-db",
"client/statement-store",
"client/storage-monitor",
"client/sysinfo",
"client/sync-state-rpc",
Expand Down Expand Up @@ -150,6 +152,7 @@ members = [
"frame/sudo",
"frame/root-offences",
"frame/root-testing",
"frame/statement",
"frame/support",
"frame/support/procedural",
"frame/support/procedural/tools",
Expand Down Expand Up @@ -219,6 +222,7 @@ members = [
"primitives/session",
"primitives/staking",
"primitives/state-machine",
"primitives/statement-store",
"primitives/std",
"primitives/storage",
"primitives/test-primitives",
Expand Down
1 change: 1 addition & 0 deletions bin/node-template/node/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ sc-telemetry = { version = "4.0.0-dev", path = "../../../client/telemetry" }
sc-keystore = { version = "4.0.0-dev", path = "../../../client/keystore" }
sc-transaction-pool = { version = "4.0.0-dev", path = "../../../client/transaction-pool" }
sc-transaction-pool-api = { version = "4.0.0-dev", path = "../../../client/transaction-pool/api" }
sc-statement-store = { version = "4.0.0-dev", path = "../../../client/statement-store" }
sc-consensus-aura = { version = "0.10.0-dev", path = "../../../client/consensus/aura" }
sp-consensus-aura = { version = "0.10.0-dev", path = "../../../primitives/consensus/aura" }
sp-consensus = { version = "0.10.0-dev", path = "../../../primitives/consensus/common" }
Expand Down
2 changes: 2 additions & 0 deletions bin/node/cli/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,11 @@ sc-chain-spec = { version = "4.0.0-dev", path = "../../../client/chain-spec" }
sc-consensus = { version = "0.10.0-dev", path = "../../../client/consensus/common" }
sc-transaction-pool = { version = "4.0.0-dev", path = "../../../client/transaction-pool" }
sc-transaction-pool-api = { version = "4.0.0-dev", path = "../../../client/transaction-pool/api" }
sc-statement-store = { version = "4.0.0-dev", path = "../../../client/statement-store" }
sc-network = { version = "0.10.0-dev", path = "../../../client/network" }
sc-network-common = { version = "0.10.0-dev", path = "../../../client/network/common" }
sc-network-sync = { version = "0.10.0-dev", path = "../../../client/network/sync" }
sc-network-statement = { version = "0.10.0-dev", path = "../../../client/network/statement" }
sc-consensus-slots = { version = "0.10.0-dev", path = "../../../client/consensus/slots" }
sc-consensus-babe = { version = "0.10.0-dev", path = "../../../client/consensus/babe" }
grandpa = { version = "0.10.0-dev", package = "sc-consensus-grandpa", path = "../../../client/consensus/grandpa" }
Expand Down
3 changes: 2 additions & 1 deletion bin/node/cli/benches/block_production.rs
Original file line number Diff line number Diff line change
Expand Up @@ -108,7 +108,8 @@ fn new_node(tokio_handle: Handle) -> node_cli::service::NewFullBase {
max_runtime_instances: 8,
runtime_cache_size: 2,
announce_block: true,
base_path: Some(base_path),
data_path: base_path.path().into(),
base_path,
informant_output_format: Default::default(),
wasm_runtime_overrides: None,
};
Expand Down
3 changes: 2 additions & 1 deletion bin/node/cli/benches/transaction_pool.rs
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,8 @@ fn new_node(tokio_handle: Handle) -> node_cli::service::NewFullBase {
max_runtime_instances: 8,
runtime_cache_size: 2,
announce_block: true,
base_path: Some(base_path),
data_path: base_path.path().into(),
base_path,
informant_output_format: Default::default(),
wasm_runtime_overrides: None,
};
Expand Down
49 changes: 46 additions & 3 deletions bin/node/cli/src/service.rs
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@ use sc_network::{event::Event, NetworkEventStream, NetworkService};
use sc_network_common::sync::warp::WarpSyncParams;
use sc_network_sync::SyncingService;
use sc_service::{config::Configuration, error::Error as ServiceError, RpcHandlers, TaskManager};
use sc_statement_store::Store as StatementStore;
use sc_telemetry::{Telemetry, TelemetryWorker};
use sp_api::ProvideRuntimeApi;
use sp_core::crypto::Pair;
Expand Down Expand Up @@ -148,6 +149,7 @@ pub fn new_partial(
),
grandpa::SharedVoterState,
Option<Telemetry>,
Arc<StatementStore>,
),
>,
ServiceError,
Expand Down Expand Up @@ -227,6 +229,15 @@ pub fn new_partial(

let import_setup = (block_import, grandpa_link, babe_link);

let statement_store = sc_statement_store::Store::new_shared(
&config.data_path,
Default::default(),
client.clone(),
config.prometheus_registry(),
&task_manager.spawn_handle(),
)
.map_err(|e| ServiceError::Other(format!("Statement store error: {:?}", e)))?;

let (rpc_extensions_builder, rpc_setup) = {
let (_, grandpa_link, _) = &import_setup;

Expand All @@ -247,6 +258,7 @@ pub fn new_partial(
let chain_spec = config.chain_spec.cloned_box();

let rpc_backend = backend.clone();
let rpc_statement_store = statement_store.clone();
let rpc_extensions_builder = move |deny_unsafe, subscription_executor| {
let deps = node_rpc::FullDeps {
client: client.clone(),
Expand All @@ -265,6 +277,7 @@ pub fn new_partial(
subscription_executor,
finality_provider: finality_proof_provider.clone(),
},
statement_store: rpc_statement_store.clone(),
};

node_rpc::create_full(deps, rpc_backend.clone()).map_err(Into::into)
Expand All @@ -281,7 +294,7 @@ pub fn new_partial(
select_chain,
import_queue,
transaction_pool,
other: (rpc_extensions_builder, import_setup, rpc_setup, telemetry),
other: (rpc_extensions_builder, import_setup, rpc_setup, telemetry, statement_store),
})
}

Expand Down Expand Up @@ -325,7 +338,7 @@ pub fn new_full_base(
keystore_container,
select_chain,
transaction_pool,
other: (rpc_builder, import_setup, rpc_setup, mut telemetry),
other: (rpc_builder, import_setup, rpc_setup, mut telemetry, statement_store),
} = new_partial(&config)?;

let shared_voter_state = rpc_setup;
Expand All @@ -335,6 +348,16 @@ pub fn new_full_base(
&config.chain_spec,
);

let statement_handler_proto = sc_network_statement::StatementHandlerPrototype::new(
client
.block_hash(0u32.into())
.ok()
.flatten()
.expect("Genesis block exists; qed"),
config.chain_spec.fork_id(),
);
config.network.extra_sets.push(statement_handler_proto.set_config());

config
.network
.extra_sets
Expand Down Expand Up @@ -526,7 +549,7 @@ pub fn new_full_base(
sync: Arc::new(sync_service.clone()),
telemetry: telemetry.as_ref().map(|x| x.handle()),
voting_rule: grandpa::VotingRulesBuilder::default().build(),
prometheus_registry,
prometheus_registry: prometheus_registry.clone(),
shared_voter_state,
};

Expand All @@ -539,6 +562,26 @@ pub fn new_full_base(
);
}

// Spawn statement protocol worker
let statement_protocol_executor = {
let spawn_handle = task_manager.spawn_handle();
Box::new(move |fut| {
spawn_handle.spawn("network-statement-validator", Some("networking"), fut);
})
};
let statement_handler = statement_handler_proto.build(
network.clone(),
sync_service.clone(),
statement_store.clone(),
prometheus_registry.as_ref(),
statement_protocol_executor,
)?;
task_manager.spawn_handle().spawn(
"network-statement-handler",
Some("networking"),
statement_handler.run(),
);

network_starter.start_network();
Ok(NewFullBase {
task_manager,
Expand Down
1 change: 1 addition & 0 deletions bin/node/executor/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ sp-keystore = { version = "0.13.0", path = "../../../primitives/keystore" }
sp-state-machine = { version = "0.13.0", path = "../../../primitives/state-machine" }
sp-tracing = { version = "6.0.0", path = "../../../primitives/tracing" }
sp-trie = { version = "7.0.0", path = "../../../primitives/trie" }
sp-statement-store = { version = "4.0.0-dev", path = "../../../primitives/statement-store" }

[dev-dependencies]
criterion = "0.4.0"
Expand Down
5 changes: 4 additions & 1 deletion bin/node/executor/src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,10 @@ pub use sc_executor::NativeElseWasmExecutor;
pub struct ExecutorDispatch;

impl sc_executor::NativeExecutionDispatch for ExecutorDispatch {
type ExtendHostFunctions = frame_benchmarking::benchmarking::HostFunctions;
type ExtendHostFunctions = (
frame_benchmarking::benchmarking::HostFunctions,
sp_statement_store::runtime_api::HostFunctions,
);

fn dispatch(method: &str, data: &[u8]) -> Option<Vec<u8>> {
kitchensink_runtime::api::dispatch(method, data)
Expand Down
1 change: 1 addition & 0 deletions bin/node/rpc/Cargo.toml
Original file line number Diff line number Diff line change
Expand Up @@ -35,5 +35,6 @@ sp-consensus = { version = "0.10.0-dev", path = "../../../primitives/consensus/c
sp-consensus-babe = { version = "0.10.0-dev", path = "../../../primitives/consensus/babe" }
sp-keystore = { version = "0.13.0", path = "../../../primitives/keystore" }
sp-runtime = { version = "7.0.0", path = "../../../primitives/runtime" }
sp-statement-store = { version = "4.0.0-dev", path = "../../../primitives/statement-store" }
substrate-frame-rpc-system = { version = "4.0.0-dev", path = "../../../utils/frame/rpc/system" }
substrate-state-trie-migration-rpc = { version = "4.0.0-dev", path = "../../../utils/frame/rpc/state-trie-migration-rpc/" }
Loading