Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Added a new application method for sync from blockvault #9749

Merged
merged 8 commits into from
Dec 5, 2020
Merged

Added a new application method for sync from blockvault #9749

merged 8 commits into from
Dec 5, 2020

Conversation

ndcgundlach
Copy link
Contributor

Change Description

Added a new application method for blockvault to use for syncing. It avoids a lot logic that is not necessary when syncing externally.

Change Type

Select ONE

  • Documentation
  • Stability bug fix
  • [ x] Other
  • Other - special case

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

heifner
heifner previously requested changes Dec 5, 2020
View reviewed changes
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
plugins/producer_plugin/producer_plugin.cpp Outdated Show resolved Hide resolved
plugins/chain_plugin/include/eosio/chain_plugin/blockvault_sync_strategy.hpp Outdated Show resolved Hide resolved
@ndcgundlach ndcgundlach marked this pull request as ready for review December 5, 2020 21:47
@ndcgundlach ndcgundlach merged commit f5c9db3 into develop Dec 5, 2020
@kj4ezj kj4ezj deleted the bv_add_new_sync branch July 19, 2021 17:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@heifner heifner heifner left review comments

@b1bart b1bart b1bart approved these changes

@larryk85 larryk85 Awaiting requested review from larryk85

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@ndcgundlach @heifner @larryk85 @b1bart