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

Reply via email to