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

Remove new block id notify feature - develop - #8498

Merged
heifner merged 4 commits into
developfrom
block-id-notify-revert-dev
Jan 25, 2020
Merged

heifner merged 4 commits into
developfrom
block-id-notify-revert-dev

Conversation

@heifner

@heifner heifner commented Jan 23, 2020

Copy link
Copy Markdown
Contributor

Change Description

  • Remove new block id notify feature as it doesn't actually improve performance and actually causes issues with large # of connections.
  • Since this was the only new net protocol feature added in 2.0, the net protocol version has been reverted to 1.8 .. 1.0 version.
  • Do not send blocks when syncing with peer.
  • Fix first sync asking for less than lib.

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@heifner
heifner merged commit 37d900e into develop Jan 25, 2020
@heifner
heifner deleted the block-id-notify-revert-dev branch January 25, 2020 03:49
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.

2 participants