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

Drop invalid messages and set low priority #804

Merged
merged 1 commit into from Jun 9, 2019

Conversation

3 participants
@vncoelho
Copy link
Member

commented Jun 9, 2019

I will open this PR as a draft until we further discuss it.

We believe that some problem with priorities and dropping might be happening.

@vncoelho

This comment has been minimized.

Copy link
Member Author

commented Jun 9, 2019

We are considering a minimum UT for the P2P, @igormcoelho, for checking this.

@vncoelho vncoelho added the port-to-3.x label Jun 9, 2019

@shargon

shargon approved these changes Jun 9, 2019

@shargon shargon marked this pull request as ready for review Jun 9, 2019

@igormcoelho
Copy link
Contributor

left a comment

tests are coming soon

@vncoelho vncoelho merged commit 8a87577 into master-2.x Jun 9, 2019

1 check passed

continuous-integration/travis-ci/push The Travis CI build passed
Details

@vncoelho vncoelho deleted the drop-msg-set-low-priority branch Jun 9, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.