[
https://issues.apache.org/jira/browse/DISPATCH-1237?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16735313#comment-16735313
]
ASF GitHub Bot commented on DISPATCH-1237:
------------------------------------------
codecov-io commented on issue #436: DISPATCH-1237 - Modified link cleanup code
to not release a multicast…
URL: https://github.com/apache/qpid-dispatch/pull/436#issuecomment-451772267
#
[Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/436?src=pr&el=h1)
Report
> Merging
[#436](https://codecov.io/gh/apache/qpid-dispatch/pull/436?src=pr&el=desc) into
[master](https://codecov.io/gh/apache/qpid-dispatch/commit/ee0d77a57feae97ba21fcfda6f65ac06b29226e5?src=pr&el=desc)
will **decrease** coverage by `0.03%`.
> The diff coverage is `66.66%`.
[](https://codecov.io/gh/apache/qpid-dispatch/pull/436?src=pr&el=tree)
```diff
@@ Coverage Diff @@
## master #436 +/- ##
==========================================
- Coverage 86.97% 86.93% -0.04%
==========================================
Files 85 85
Lines 18699 18705 +6
==========================================
- Hits 16264 16262 -2
- Misses 2435 2443 +8
```
| [Impacted
Files](https://codecov.io/gh/apache/qpid-dispatch/pull/436?src=pr&el=tree) |
Coverage Δ | |
|---|---|---|
|
[src/router\_core/connections.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2Nvbm5lY3Rpb25zLmM=)
| `94.9% <66.66%> (-0.35%)` | :arrow_down: |
|
[src/container.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL2NvbnRhaW5lci5j)
| `77.4% <0%> (-0.76%)` | :arrow_down: |
|
[src/router\_core/agent\_link.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2FnZW50X2xpbmsuYw==)
| `63.84% <0%> (-0.57%)` | :arrow_down: |
|
[src/router\_core/transfer.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL3RyYW5zZmVyLmM=)
| `90.14% <0%> (-0.45%)` | :arrow_down: |
|
[src/iterator.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL2l0ZXJhdG9yLmM=)
| `93.45% <0%> (-0.22%)` | :arrow_down: |
|
[src/router\_node.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9ub2RlLmM=)
| `93.95% <0%> (-0.13%)` | :arrow_down: |
|
[src/remote\_sasl.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JlbW90ZV9zYXNsLmM=)
| `83.52% <0%> (+0.28%)` | :arrow_up: |
|
[src/router\_core/core\_client\_api.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2NvcmVfY2xpZW50X2FwaS5j)
| `92.67% <0%> (+0.36%)` | :arrow_up: |
|
[src/router\_core/forwarder.c](https://codecov.io/gh/apache/qpid-dispatch/pull/436/diff?src=pr&el=tree#diff-c3JjL3JvdXRlcl9jb3JlL2ZvcndhcmRlci5j)
| `93.92% <0%> (+0.7%)` | :arrow_up: |
------
[Continue to review full report at
Codecov](https://codecov.io/gh/apache/qpid-dispatch/pull/436?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/436?src=pr&el=footer).
Last update
[ee0d77a...52e4660](https://codecov.io/gh/apache/qpid-dispatch/pull/436?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 GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
> large frame multicast can stall receivers if one receiver closes its
> connection
> -------------------------------------------------------------------------------
>
> Key: DISPATCH-1237
> URL: https://issues.apache.org/jira/browse/DISPATCH-1237
> Project: Qpid Dispatch
> Issue Type: Bug
> Components: Router Node
> Environment: The attached patch updates the edge router test to
> reproduce this issue.
> NOTE WELL - you must have the fix for DISPATCH-1235 applied before running
> this test else the router will crash.
> Reporter: Ken Giusti
> Assignee: Ganesh Murthy
> Priority: Major
> Fix For: 1.5.0
>
> Attachments: test.patch
>
>
> Ran the following scenario on an edge router:
> 1 sender (not presettled)
> 3 receivers
> multicast
> large message
> If I have one of the receivers close its connection while in the middle of a
> multi frame transfer the router marks the message as "discard" and does not
> forward the remaining frames to the two remaining clients, thus stalling
> their links.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
