grs commented on a change in pull request #858: URL: https://github.com/apache/qpid-dispatch/pull/858#discussion_r499788668
########## File path: src/adaptors/http1/http1_client.c ########## @@ -1202,8 +1220,11 @@ uint64_t qdr_http1_client_core_link_deliver(qdr_http1_adaptor_t *adaptor, _client_response_msg_t *rmsg = DEQ_TAIL(hreq->responses); assert(rmsg && rmsg->dlv == delivery); - if (!rmsg->dispo) + if (!rmsg->dispo) { rmsg->dispo = _encode_response_message(hreq, rmsg); + hreq->base.stop = qd_timer_now(); Review comment: Not sure what you mean by INFO responses? The point of the start and stop times is just to compute the latency for the request so we can track the maximum latency seen over time. ---------------------------------------------------------------- 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: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@qpid.apache.org For additional commands, e-mail: dev-h...@qpid.apache.org