Sitelet https://github.com/EOSIO/eos/pull/9752
Skip to content
This repository was archived by the owner on Aug 2, 2022. It is now read-only.

sync from block vault during block production - #9752

Merged
huangminghuang merged 6 commits into
developfrom
huangminghuang/bv-sync
Dec 7, 2020
Merged

huangminghuang merged 6 commits into
developfrom
huangminghuang/bv-sync

Conversation

@huangminghuang

Copy link
Copy Markdown
Contributor

Change Description

This PR allows nodeos to sync with block vault when it doesn't receive the required block from peer logical producer node within 1 second.

Change Type

Select ONE

  • Documentation
  • Stability bug fix
  • Other
  • Other - special case

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@huangminghuang
huangminghuang requested a review from b1bart December 5, 2020 22:45
@nksanthosh
nksanthosh requested a review from heifner December 6, 2020 01:25
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
@huangminghuang
huangminghuang requested a review from b1bart December 6, 2020 03:39
try {
_impl->blockvault->sync(&id, *this);
} catch( fc::exception& er ) {
wlog("Attempting to resync from blockvault encountered ${details}; the node must restart to "

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should be fc_wlog.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You missed:
line 326 dlog.
line 341 elog

Not sure if this was intentional.

Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
Comment thread plugins/producer_plugin/producer_plugin.cpp Outdated
# Conflicts:
#	tests/blockvault_tests.py
@huangminghuang
huangminghuang merged commit 56d9c78 into develop Dec 7, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants