[jira] [Commented] (PROTON-1536) THere is no way using the C++ binding connection_driver API to either send heartbeat frames, or recognise heartbeat timeouts

classic Classic list List threaded Threaded
1 message Options
Reply | Threaded
Open this post in threaded view
|  
Report Content as Inappropriate

[jira] [Commented] (PROTON-1536) THere is no way using the C++ binding connection_driver API to either send heartbeat frames, or recognise heartbeat timeouts

JIRA jira@apache.org

    [ https://issues.apache.org/jira/browse/PROTON-1536?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16123741#comment-16123741 ]

Andrew Stitcher commented on PROTON-1536:
-----------------------------------------

This code change has very rudimentary tests. Just one positive and one negative test.

Specifically it doesn't test heartbeat frame generation. Although it should be straightforward to force generation of heartbeat frames - I think it might be difficult to recognise they have/have not arrived.

> THere is no way using the C++ binding connection_driver API to either send heartbeat frames, or recognise heartbeat timeouts
> ----------------------------------------------------------------------------------------------------------------------------
>
>                 Key: PROTON-1536
>                 URL: https://issues.apache.org/jira/browse/PROTON-1536
>             Project: Qpid Proton
>          Issue Type: Improvement
>          Components: cpp-binding
>            Reporter: Andrew Stitcher
>            Assignee: Andrew Stitcher
>             Fix For: proton-c-0.18.0
>
>
> The C++ connection_driver API does not expose any way to get to the pn_transport_tick() function to tell the engine what time it is. So the engine has no way to know whether it needs to send empty heartbeat frames, and equally it can't tell when a heartbeat timeout has occurred due to lack of traffic from the peer.



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)

---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Loading...