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

Confirmation height enhancement #1801

Merged

Conversation

@SergiySW
Copy link
Collaborator

commented Mar 3, 2019

  • improve add_confirmation_heights ()
  • disable process_fork () for confirmed blocks
  • replace active blocks check with confirmation status in RPCs pending*
  • replace election::confirm_back () with add_confirmation_heights ()
  • simplify search_pending for confirmed blocks
  • merge RPC block_confirmed into block_info & blocks_info

@SergiySW SergiySW added the enhancement label Mar 3, 2019

@SergiySW SergiySW added this to the V19.0 milestone Mar 3, 2019

@SergiySW SergiySW self-assigned this Mar 3, 2019

@SergiySW SergiySW requested a review from cryptocode Mar 3, 2019

@SergiySW SergiySW added this to CP1 (2019-03-13) in V19 Mar 3, 2019

@zhyatt zhyatt requested review from wezrule and clemahieu Mar 4, 2019

@SergiySW SergiySW requested a review from argakiig Mar 5, 2019

@wezrule

wezrule approved these changes Mar 6, 2019

@SergiySW SergiySW merged commit 8f4c02c into nanocurrency:master Mar 9, 2019

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@zhyatt zhyatt added the documentation label May 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.