[
https://issues.apache.org/jira/browse/DISPATCH-872?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16300220#comment-16300220
]
ASF GitHub Bot commented on DISPATCH-872:
-----------------------------------------
Github user ted-ross commented on a diff in the pull request:
https://github.com/apache/qpid-dispatch/pull/226#discussion_r158315973
--- Diff: tests/system_tests_one_router.py ---
@@ -1552,6 +1552,14 @@ def check_if_done(self):
self.conn.close()
elif self.n_sent == self.n_released:
self.error = None
+
+ if not self.error:
--- End diff --
This 'if' is unnecessary. Also, this test doesn't send pre-settled
deliveries. Why does it expect a count of dropped pre-settleds?
> Add a counter for dropped-presettleds on links
> ----------------------------------------------
>
> Key: DISPATCH-872
> URL: https://issues.apache.org/jira/browse/DISPATCH-872
> Project: Qpid Dispatch
> Issue Type: Improvement
> Components: Router Node
> Reporter: Ted Ross
> Assignee: Ganesh Murthy
> Fix For: 1.1.0
>
>
> When the router drops pre-settled deliveries during congestion, those drops
> are not counted or reported. A new counter should be added for links to
> account for dropped presettled deliveries.
--
This message was sent by Atlassian JIRA
(v6.4.14#64029)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
