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 2018/10/18 15:12:09 UTC
[GitHub] qpid-dispatch pull request #392: DISPATCH-1139 : address priority support an...
GitHub user mgoulish opened a pull request:
https://github.com/apache/qpid-dispatch/pull/392
DISPATCH-1139 : address priority support and test
Code that implements address-priority. Also changes things so that address priority takes precedence if present, otherwise defaults to message priority -- and messages now always have priority, defaulting to 4 if not specified. (Same as Proton default.)
Also a python test that tests both message and address priority.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/mgoulish/qpid-dispatch DISPATCH-1139-B
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/qpid-dispatch/pull/392.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 #392
----
commit bb4f3121989cb4b91d4e364287868da1594f3acf
Author: Michael Goulish <mg...@...>
Date: 2018-10-18T15:07:45Z
DISPATCH-1139 : address priority support and test
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] qpid-dispatch pull request #392: DISPATCH-1139 : address priority support an...
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/qpid-dispatch/pull/392
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org
[GitHub] qpid-dispatch issue #392: DISPATCH-1139 : address priority support and test
Posted by codecov-io <gi...@git.apache.org>.
Github user codecov-io commented on the issue:
https://github.com/apache/qpid-dispatch/pull/392
# [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=h1) Report
> Merging [#392](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=desc) into [master](https://codecov.io/gh/apache/qpid-dispatch/commit/9c5666f6d1e36979d6d4b3e657c5c5246ffbf5df?src=pr&el=desc) will **increase** coverage by `0.04%`.
> The diff coverage is `85.71%`.
[![Impacted file tree graph](https://codecov.io/gh/apache/qpid-dispatch/pull/392/graphs/tree.svg?width=650&token=rk2Cgd27pP&height=150&src=pr)](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #392 +/- ##
==========================================
+ Coverage 85% 85.04% +0.04%
==========================================
Files 76 76
Lines 16770 16783 +13
==========================================
+ Hits 14255 14273 +18
+ Misses 2515 2510 -5
```
| [Impacted Files](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=tree) | Coverage Δ | |
|---|---|---|
| [src/router\_core/forwarder.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2ZvcndhcmRlci5j) | `93.25% <100%> (-0.05%)` | :arrow_down: |
| [src/router\_core/route\_control.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX2NvbnRyb2wuYw==) | `95.9% <100%> (ø)` | :arrow_up: |
| [src/router\_core/core\_link\_endpoint.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2NvcmVfbGlua19lbmRwb2ludC5j) | `94.44% <100%> (ø)` | :arrow_up: |
| [src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=) | `95.24% <100%> (+0.23%)` | :arrow_up: |
| [src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=) | `93.04% <100%> (-0.28%)` | :arrow_down: |
| [src/message.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL21lc3NhZ2UuYw==) | `88.69% <100%> (+0.39%)` | :arrow_up: |
| [src/router\_config.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb25maWcuYw==) | `96.32% <100%> (+0.04%)` | :arrow_up: |
| [src/router\_core/agent\_config\_address.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2NvbmZpZ19hZGRyZXNzLmM=) | `78.11% <60%> (-0.72%)` | :arrow_down: |
| [src/router\_core/core\_timer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2NvcmVfdGltZXIuYw==) | `94.64% <0%> (-1.79%)` | :arrow_down: |
| ... and [7 more](https://codecov.io/gh/apache/qpid-dispatch/pull/392/diff?src=pr&el=tree-more) | |
------
[Continue to review full report at Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=continue).
> **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
> Powered by [Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=footer). Last update [9c5666f...bb4f312](https://codecov.io/gh/apache/qpid-dispatch/pull/392?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@qpid.apache.org
For additional commands, e-mail: dev-help@qpid.apache.org