grs commented on a change in pull request #897:
URL: https://github.com/apache/qpid-dispatch/pull/897#discussion_r512855336
##########
File path: src/adaptors/http2/http2_adaptor.h
##########
@@ -138,6 +141,7 @@ struct qdr_http2_connection_t {
bool ingress;
bool timer_scheduled;
bool client_magic_sent;
+ time_t prev_ping; // Time the previous PING frame was
sent on egress connection.
Review comment:
Right, and that is fine. All I'm saying is that if you send out any
other frame within the second, there is no need to send a ping. However that is
an optimisation. It is good as is.
----------------------------------------------------------------
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:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
