You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@qpid.apache.org by mgoulish <gi...@git.apache.org> on 2017/05/10 13:38:36 UTC
[GitHub] qpid-dispatch pull request #162: DISPATCH-209 -- revamp of second three-rout...
GitHub user mgoulish opened a pull request:
https://github.com/apache/qpid-dispatch/pull/162
DISPATCH-209 -- revamp of second three-router test.
This new version of three-router test #2 survived a 1000-iteration test with no failures.
The main changes are
1. handle on_sendable event and the sending of messages better. (Before I was only sending 1 message per on_sendable event, instead of sending until credit exhausteed.)
2. indirectly detect when the network is not yet ready for me to send (in spite of on_sendable event) and pause for 1 second to allow it to propagate endpoint info everywhere.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mgoulish/qpid-dispatch master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/qpid-dispatch/pull/162.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #162
----
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] qpid-dispatch pull request #162: DISPATCH-209 -- revamp of second three-rout...
Posted by mgoulish <gi...@git.apache.org>.
Github user mgoulish closed the pull request at:
https://github.com/apache/qpid-dispatch/pull/162
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] qpid-dispatch issue #162: DISPATCH-209 -- revamp of second three-router test...
Posted by mgoulish <gi...@git.apache.org>.
Github user mgoulish commented on the issue:
https://github.com/apache/qpid-dispatch/pull/162
closing to replace with new PR containing 3rd test.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org