[
https://issues.apache.org/jira/browse/DISPATCH-1428?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16933635#comment-16933635
]
ASF GitHub Bot commented on DISPATCH-1428:
------------------------------------------
codecov-io commented on issue #567: DISPATCH-1428
URL: https://github.com/apache/qpid-dispatch/pull/567#issuecomment-533213932
#
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/567?src=pr&el=h1)
Report
> Merging
[#567](https://codecov.io/gh/apache/qpid-dispatch/pull/567?src=pr&el=desc) into
[master](https://codecov.io/gh/apache/qpid-dispatch/commit/759dfd6be57c2d32a371b1529194831356de61c5?src=pr&el=desc)
will **decrease** coverage by `0.08%`.
> The diff coverage is `97.05%`.
[](https://codecov.io/gh/apache/qpid-dispatch/pull/567?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #567 +/- ##
==========================================
- Coverage 86.35% 86.27% -0.09%
==========================================
Files 90 90
Lines 20364 20382 +18
==========================================
- Hits 17585 17584 -1
- Misses 2779 2798 +19
```
| [Impacted
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/567?src=pr&el=tree) |
Coverage Δ | |
|---|---|---|
|
[src/router\_core/route\_control.c](https://codecov.io/gh/apache/qpid-dispatch/pull/567/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlX2NvbnRyb2wuYw==)
| `94.03% <97.05%> (-3.27%)` | :arrow_down: |
|
[src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/567/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
| `93.7% <0%> (-0.76%)` | :arrow_down: |
|
[src/router\_node.c](https://codecov.io/gh/apache/qpid-dispatch/pull/567/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
| `92.77% <0%> (-0.37%)` | :arrow_down: |
|
[src/parse.c](https://codecov.io/gh/apache/qpid-dispatch/pull/567/diff?src=pr&el=tree#diff-c3JjL3BhcnNlLmM=)
| `87.64% <0%> (-0.23%)` | :arrow_down: |
|
[src/router\_core/router\_core.c](https://codecov.io/gh/apache/qpid-dispatch/pull/567/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3JvdXRlcl9jb3JlLmM=)
| `86.53% <0%> (-0.2%)` | :arrow_down: |
|
[src/iterator.c](https://codecov.io/gh/apache/qpid-dispatch/pull/567/diff?src=pr&el=tree#diff-c3JjL2l0ZXJhdG9yLmM=)
| `89.32% <0%> (+0.16%)` | :arrow_up: |
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/567?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/567?src=pr&el=footer).
Last update
[759dfd6...71b5a8a](https://codecov.io/gh/apache/qpid-dispatch/pull/567?src=pr&el=lastupdated).
Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> route connection not indexed by 'connection' field of connector
> ---------------------------------------------------------------
>
> Key: DISPATCH-1428
> URL: https://issues.apache.org/jira/browse/DISPATCH-1428
> Project: Qpid Dispatch
> Issue Type: Bug
> Reporter: Gordon Sim
> Priority: Major
>
> The connection established by a route-container connector will not be indexed
> by the 'connection' field of that connector if there is already an existing
> route-connection with the same container id but established by a different
> connector.
> E.g. start router on 5672 and a broker (or a separate router emulating a
> broker) on 5673, then:
> {noformat}
> for n in foo bar; do
> qdmanage CREATE --type connector --name $n role=route-container
> host=localhost port=5673;
> qdmanage CREATE --type linkRoute --name $n pattern=$n direction=in
> connection=$n;
> done;
> qdstat --linkroute
> {noformat}
> Only one of these link routes is active though both connections are
> established.
> The issues is that when the first connection is established, it indexes the
> qdr_conn_identifier_t by container-id and the connection label from the
> connector. When the second connection is established, it looks up first by
> container id, so adds itself to the qdr_conn_identifier_t created by the
> first connection. There is then no entry in the index for the connection
> label of the second connector, so the second link route can never be
> activated.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org
For additional commands, e-mail: dev-h...@qpid.apache.org
