[GitHub] qpid-dispatch issue #389: DISPATCH-1130 Expose link priority

2018-10-10 Thread ganeshmurthy 
Github user ganeshmurthy commented on the issue:

https://github.com/apache/qpid-dispatch/pull/389
  
@ErnieAllen  This change looks good. As part of this PR, can you also 
please add a test to system_tests_qdstat to verify that the correct link 
priority is showing up? Thanks.


---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org

[GitHub] qpid-dispatch issue #389: DISPATCH-1130 Expose link priority

2018-10-10 Thread codecov-io 
Github user codecov-io commented on the issue:

https://github.com/apache/qpid-dispatch/pull/389
  
# 
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=h1) 
Report
> Merging 
[#389](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=desc) into 
[master](https://codecov.io/gh/apache/qpid-dispatch/commit/d1d6e92df2d3adbba5d86e650e0d44a0ffc0b33e?src=pr=desc)
 will **decrease** coverage by `0.03%`.
> The diff coverage is `100%`.

[![Impacted file tree 
graph](https://codecov.io/gh/apache/qpid-dispatch/pull/389/graphs/tree.svg?width=650=rk2Cgd27pP=150=pr)](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=tree)

```diff
@@Coverage Diff @@
##   master #389  +/-   ##
==
- Coverage   85.06%   85.02%   -0.04% 
==
  Files  73   73  
  Lines   1652316525   +2 
==
- Hits1405514051   -4 
- Misses   2468 2474   +6
```


| [Impacted 
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=tree) | 
Coverage Δ | |
|---|---|---|
| 
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/389/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
 | `63.63% <100%> (+0.41%)` | :arrow_up: |
| 
[src/router\_core/core\_timer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/389/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL2NvcmVfdGltZXIuYw==)
 | `94.64% <0%> (-1.79%)` | :arrow_down: |
| 
[src/router\_core/modules/core\_test\_hooks.c](https://codecov.io/gh/apache/qpid-dispatch/pull/389/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL21vZHVsZXMvY29yZV90ZXN0X2hvb2tzLmM=)
 | `89.54% <0%> (-1.37%)` | :arrow_down: |
| 
[src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/389/diff?src=pr=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
 | `92.05% <0%> (-0.34%)` | :arrow_down: |
| 
[src/container.c](https://codecov.io/gh/apache/qpid-dispatch/pull/389/diff?src=pr=tree#diff-c3JjL2NvbnRhaW5lci5j)
 | `76.55% <0%> (-0.2%)` | :arrow_down: |

--

[Continue to review full report at 
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=continue).
> **Legend** - [Click here to learn 
more](https://docs.codecov.io/docs/codecov-delta)
> `Δ = absolute  (impact)`, `ø = not affected`, `? = missing 
data`
> Powered by 
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=footer).
 Last update 
[d1d6e92...a81784b](https://codecov.io/gh/apache/qpid-dispatch/pull/389?src=pr=lastupdated).
 Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).



---

-
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org