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

Feature/fast sync update #1284

Merged
merged 61 commits into from
Sep 11, 2022
Merged

Feature/fast sync update #1284

merged 61 commits into from
Sep 11, 2022

Conversation

sanblch
Copy link
Contributor

@sanblch sanblch commented Jul 27, 2022

Referenced issues

#1164

Description of the Change

Adds /dot/state/2 protocol
Adds fast syn functionality

Benefits

--sync Fast gives faster synchronization option

Possible Drawbacks

not well tested behavior on peer response error at state syncing
restart of synced node with fast sync option is undefined

Usage Examples or Tests

Alternate Designs

@sanblch sanblch force-pushed the feature/fast-sync-update branch from afeab7b to 71a8463 Compare July 28, 2022 05:19
@sanblch sanblch marked this pull request as ready for review July 28, 2022 06:04
@sanblch sanblch requested review from xDimon and Harrm July 28, 2022 06:04
@codecov
Copy link

codecov bot commented Jul 28, 2022

Codecov Report

Merging #1284 (82452cb) into master (e341962) will decrease coverage by 0.39%.
The diff coverage is 5.67%.

❗ Current head 82452cb differs from pull request most recent head 866a6d0. Consider uploading reports for the commit 866a6d0 to get more accurate results

@@            Coverage Diff             @@
##           master    #1284      +/-   ##
==========================================
- Coverage   25.36%   24.97%   -0.40%     
==========================================
  Files         607      610       +3     
  Lines       22109    22553     +444     
  Branches    11490    11779     +289     
==========================================
+ Hits         5609     5633      +24     
- Misses      11328    11718     +390     
- Partials     5172     5202      +30     
Impacted Files Coverage Δ
core/application/app_configuration.hpp 100.00% <ø> (ø)
core/application/impl/app_configuration_impl.hpp 28.37% <0.00%> (-0.79%) ⬇️
core/consensus/babe/babe.hpp 100.00% <ø> (ø)
core/consensus/babe/impl/babe_impl.hpp 100.00% <ø> (ø)
core/consensus/grandpa/impl/environment_impl.cpp 11.88% <ø> (+0.11%) ⬆️
core/consensus/grandpa/impl/grandpa_impl.cpp 2.64% <0.00%> (-0.08%) ⬇️
core/consensus/grandpa/impl/grandpa_impl.hpp 100.00% <ø> (ø)
core/consensus/grandpa/impl/voting_round_impl.hpp 36.36% <ø> (ø)
core/log/configurator.cpp 22.22% <ø> (ø)
...ore/network/impl/protocols/state_protocol_impl.cpp 0.00% <0.00%> (ø)
... and 17 more

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sanblch sanblch force-pushed the feature/fast-sync-update branch from 71a8463 to 760fef3 Compare August 4, 2022 14:22
xDimon and others added 12 commits August 4, 2022 18:13
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: iceseer <iceseer@gmail.com>
* feature: change logging level over RPC

Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
xDimon added 7 commits August 15, 2022 16:26
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
@xDimon xDimon requested a review from kamilsa September 8, 2022 14:12
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
Signed-off-by: Dmitriy Khaustov aka xDimon <khaustov.dm@gmail.com>
@xDimon xDimon enabled auto-merge (squash) September 11, 2022 05:27
@xDimon xDimon merged commit 0c3398f into master Sep 11, 2022
@xDimon xDimon deleted the feature/fast-sync-update branch September 11, 2022 05:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants