Piper/full request to response await api #1
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
p2p: Correctly handle peers that disconnect before sending the auth ack
p2p: Make it easy for anyone to run LightPeerChain's integration tests
p2p: Log more details on peer authentication errors
In regular-sync mode, there was a bug causing it to keep waiting even after completing a sync with a given peer, but instead it should return so that FastChainSyncer can trigger another sync the next time it is notified about a new block
p2p: FastChainSyncer._sync() always returns when sync is complete
p2p: Consolidate PeerPool APIs to get peers from which to fetch data
Add ERC20 deployment/interaction benchmark Relates to #592
Fix typo of nam -> name
Merkle utils
…rary convert to use CancelToken from libary
…nd-response-linking Implement stronger linkage between requests for headers and their responses
pipermerriam
force-pushed
the
piper/full-request-to-response-await-API
branch
2 times, most recently
from
f02aaa1 to
5f6bb64
Compare
…n-library-OperationCancelled-exception Switch to cancel-token library exception class
pipermerriam
force-pushed
the
piper/full-request-to-response-await-API
branch
from
3996fcd to
04c002a
Compare
…r-in-p2p-tests Use new network simulator library in tests
It will be replaced shortly with a proper fix, but should allow us to validate some assumptions and make sure the actual fix will ultimately address the issue Issue #1074
Keep track of which nodes we requested to every peer so that we know which ones they're missing when they reply with just a subset of what was requested. That way we can avoid re-requesting those missing nodes to peers that don't have them. #1073 Also, when a reply from a peer contains only a subset of what was requested, we immediately re-request the missing ones. Closes: #1077
Now if none of our peers have any of the nodes we want, those nodes are added to a list of missing node keys and retried after a while.
pipermerriam
force-pushed
the
piper/full-request-to-response-await-API
branch
from
307d1cb to
d08a4ff
Compare
…-sync-status-msg Reduce size of state sync status message
pipermerriam
force-pushed
the
piper/full-request-to-response-await-API
branch
from
d08a4ff to
b6afa5f
Compare
pipermerriam
pushed a commit
that referenced
this pull request
Oct 2, 2018
[ImgBot] optimizes images
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What was wrong?
How was it fixed?
Cute Animal Picture