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

Net plugin remove read delays - develop - #8085

Merged
heifner merged 7 commits into
developfrom
net-plugin-max
Oct 16, 2019
Merged

heifner merged 7 commits into
developfrom
net-plugin-max

Conversation

@heifner

@heifner heifner commented Oct 16, 2019

Copy link
Copy Markdown
Contributor

Change Description

  • Remove read delay timer as delaying read delays receiving blocks which introduces undesirable latency.
  • When more than max transactions in progress is reached, explicitly drop transaction with log of transaction id.
  • Also noticed during code inspection that benign_other was not being sent in case where requested block is not available. Handshake checks this case on initial handshake. However if this code is reached and block is not available, it is better to let the peer know as opposed to the peer timing out before determining a different peer is needed.
  • Also switched some error/warn level logging to info level as they are expected cases when running with a snapshot. Resolves Reduce logging level for peers in catchup to info #8052

Consensus Changes

  • Consensus Changes

API Changes

  • API Changes

Documentation Additions

  • Documentation Additions

@heifner
heifner merged commit 0940ece into develop Oct 16, 2019
@heifner
heifner deleted the net-plugin-max branch October 16, 2019 20:18
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