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

Only open a message sender when we have messages to send #4069

Merged
merged 2 commits into from
Jul 16, 2017

Conversation

whyrusleeping
Copy link
Member

resolves #3651 for the most part. Theres still a deeper fix that needs to be done to eliminate this class of errors.

License: MIT
Signed-off-by: Jeromy jeromyj@gmail.com

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
@whyrusleeping whyrusleeping added the status/in-progress In progress label Jul 16, 2017
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@whyrusleeping
Copy link
Member Author

except for all the tests failing... great.

License: MIT
Signed-off-by: Jeromy <jeromyj@gmail.com>
Copy link
Member

@Stebalien Stebalien left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM assuming the tests actually pass this time...

@whyrusleeping whyrusleeping merged commit f16d942 into master Jul 16, 2017
@whyrusleeping whyrusleeping deleted the fix/bitswap-stream-hang branch July 16, 2017 06:28
@whyrusleeping whyrusleeping removed the status/in-progress In progress label Jul 16, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Occasional stream hang
2 participants