ganeshmurthy commented on a change in pull request #897:
URL: https://github.com/apache/qpid-dispatch/pull/897#discussion_r512825599
##########
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:
The check_send_ping_frame() only conditionally updates the
conn->prev_ping field like this -
time_t current = time(NULL);
time_t prev = conn->prev_ping;
if (current - prev >= 4) {
send_ping_frame(conn);
qd_log(http2_adaptor->log_source, QD_LOG_TRACE, "[C%"PRIu64"] Sent
PING frame", conn->conn_id);
qd_timer_schedule(conn->ping_timer, 4000);
conn->prev_ping = current;
}
So the conn->prev_ping contains the time that the ping was sent out ?
----------------------------------------------------------------
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]
