-
Notifications
You must be signed in to change notification settings - Fork 61
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
libp2p: fix for slow dht lookups using extra queue #2642
Conversation
Release v0.7.1
Release v0.7.2
Release v0.7.3
Release v0.7.4
Release v0.7.5
Release v0.8.0
Release v0.9.0
Release v0.9.1
Release v0.9.2
Release v0.10.0
Release v0.10.1
c62d5da
to
a1f9dc8
Compare
493af30
to
c96dda0
Compare
Codecov Report
@@ Coverage Diff @@
## develop #2642 +/- ##
============================================
+ Coverage 99.91% 100.00% +0.08%
============================================
Files 521 516 -5
Lines 35888 35874 -14
============================================
+ Hits 35858 35874 +16
+ Misses 30 0 -30
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
27d6b9f
to
0664c3d
Compare
0664c3d
to
5d6fe9a
Compare
…2p_slow_dht_lookups
…2p_slow_dht_lookups
…2p_slow_dht_lookups
…p_slow_dht_lookups
…2p_slow_dht_lookups
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
with open(self.conn.node.log_file) as f: | ||
return "while sending slow envelope:" in f.read() | ||
|
||
wait_for_condition(_check, timeout=30, period=1) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
As I understand it, this will throw a timeout exception if the slow queue is not used, but since this is inside a try-except, won't the test always pass?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
it's try..finally to disconnect the connection in case of fail, no exceptions handled or supressed
Proposed changes
libp2p: fix for slow dht lookups using extra queue
slow dht lookup envelopes are moved to slow queue to avoid blocking of "good" envelopes
Fixes
#2634
Types of changes
Checklist
Put an
x
in the boxes that apply.develop
branch (left side). Also you should start your branch off ourdevelop
.Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...
DELETE INCLUSIVE THIS AND BELOW FOR STANDARD PR
Release summary
Version number: [e.g. 1.0.1]
Release details
Describe in short the main changes with the new release.
Checklist
Put an
x
in the boxes that apply.main
branch (left side), fromdevelop
scripts/generate_ipfs_hashes.py
)HISTORY.md
for this releaseaea/__version__.py
file.latest
(check the READMEs of
aea-develop
and
aea-user
)aea
to PyPI.Further comments
Write here any other comment about the release, if any.