Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Batch transactions in complex relays #1669

Merged
merged 32 commits into from
Dec 16, 2022
Merged
Changes from 1 commit
Commits
Show all changes
32 commits
Select commit Hold shift + click to select a range
4c3cc6b
batch transactions in message relay: API prototype
svyatonik Nov 29, 2022
e140e11
get rid of Box<dyn BatchTransaction> and actually submit it
svyatonik Nov 30, 2022
feba6fa
test batch transactions
svyatonik Nov 30, 2022
e372820
message_lane_loop_works_with_batch_transactions
svyatonik Nov 30, 2022
824fc23
removed logger
svyatonik Nov 30, 2022
f8730aa
BatchConfirmationTransaction + BatchDeliveryTransaction
svyatonik Dec 1, 2022
9cc4fe3
more prototyping
svyatonik Dec 1, 2022
14c1aa8
fmt
svyatonik Dec 1, 2022
418d6e4
continue with batch calls
svyatonik Dec 1, 2022
0dcdec8
impl BatchCallBuilder for ()
svyatonik Dec 1, 2022
d32fb4d
BatchDeliveryTransaction impl
svyatonik Dec 1, 2022
b265949
BundledBatchCallBuilder
svyatonik Dec 12, 2022
8cddd6a
Merge branch 'master' into batch-transactions-in-message-relay
svyatonik Dec 12, 2022
ac59ecc
proper impl of BundledBatchCallBuilder + use it in RialtoParachain ->…
svyatonik Dec 12, 2022
cfa95c1
impl prove_header in OnDemandHeadersRelay
svyatonik Dec 12, 2022
9e850d2
impl OnDemandParachainsRelay::prove_header (needs extensive tests)
svyatonik Dec 12, 2022
b0a277b
added a couple of TODOs
svyatonik Dec 12, 2022
a1c50d6
return Result<Option<BatchTx>> when asking for more headers
svyatonik Dec 13, 2022
b852158
prove headers when reauire_* is called && return proper headers from …
svyatonik Dec 13, 2022
38561c0
split parachains::prove_header and test select_headers_to_prove
svyatonik Dec 13, 2022
c1dff4b
more traces and leave TODOs
svyatonik Dec 14, 2022
2d8b500
use finality stream in SubstrateFinalitySource::prove_block_finality
svyatonik Dec 14, 2022
fad8069
prove parachain head at block, selected by headers relay
svyatonik Dec 14, 2022
5053b16
const ANCIENT_BLOCK_THRESHOLD
svyatonik Dec 14, 2022
b5f2623
TODO -> proof
svyatonik Dec 14, 2022
aec2063
clippy and spelling
svyatonik Dec 14, 2022
21fbb0c
Merge branch 'master' into batch-transactions-in-message-relay
svyatonik Dec 14, 2022
0dc01ab
BatchCallBuilder::build_batch_call() returns Result
svyatonik Dec 15, 2022
8370c50
Merge branch 'master' into batch-transactions-in-message-relay
svyatonik Dec 15, 2022
d1254fc
read first proof from two streams
svyatonik Dec 15, 2022
78d5214
FailedToFindFinalityProof -> FinalityProofNotFound
svyatonik Dec 16, 2022
9da39c7
changed select_headers_to_prove to version from PR review
svyatonik Dec 16, 2022
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
Prev Previous commit
Next Next commit
Merge branch 'master' into batch-transactions-in-message-relay
  • Loading branch information
svyatonik committed Dec 14, 2022
commit 21fbb0cd119ded96551eb72726dedd7baa2fe257
6 changes: 3 additions & 3 deletions relays/lib-substrate-relay/src/on_demand/parachains.rs
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ use parachains_relay::parachains_loop::{
};
use relay_substrate_client::{
AccountIdOf, AccountKeyPairOf, BlockNumberOf, CallOf, Chain, Client, Error as SubstrateError,
HashOf, HeaderIdOf, ANCIENT_BLOCK_THRESHOLD,
HashOf, HeaderIdOf, ParachainBase, ANCIENT_BLOCK_THRESHOLD,
};
use relay_utils::{
metrics::MetricsParams, relay_loop::Client as RelayClient, BlockNumberBase, FailedClient,
Expand Down Expand Up @@ -179,7 +179,7 @@ where
// despite what we've selected before (in `select_headers_to_prove` call), if headers relay
// have chose the different header (e.g. because there's no GRANDPA jusstification for it),
// we need to prove parachain head available at this header
let para_id = ParaId(P::SOURCE_PARACHAIN_PARA_ID);
let para_id = ParaId(P::SourceParachain::PARACHAIN_ID);
let mut proved_parachain_block = selected_parachain_block;
if proved_relay_block != selected_relay_block {
proved_parachain_block = parachains_source
Expand Down Expand Up @@ -618,7 +618,7 @@ impl<'a, P: SubstrateParachainsPipeline>
> for (&'a OnDemandParachainsRelay<P>, &'a ParachainsSource<P>)
{
fn parachain_id(&self) -> ParaId {
ParaId(P::SOURCE_PARACHAIN_PARA_ID)
ParaId(P::SourceParachain::PARACHAIN_ID)
}

async fn best_finalized_relay_block_at_source(
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.