paritytech / extended-parachain-template

Node template to build parachains with all the required pallets. Slightly opinionated based on what majority of parachain teams are using.
The Unlicense
27 stars 19 forks source link

Change SyncStrategy from SyncStrategy::Normal to SyncStrategy::Parachain #17

Closed weezy20 closed 1 year ago

weezy20 commented 1 year ago

https://github.com/paritytech/extended-parachain-template/blob/frontier/node/src/service.rs#L334

It's preferred to set MappingSyncWorker to use SyncStrategy::Parachain as this is a cumulus based parachain and this variant prevents "confusing" the MappingSyncWorker due to blocks being imported multiple times by cumulus and only being marked best on the second time. Relevant logic here: https://github.com/paritytech/frontier/blob/master/client/mapping-sync/src/lib.rs#L158-L159

weezy20 commented 1 year ago

Closed by #13