moonchen commented on code in PR #13667:
URL: https://github.com/apache/trafficserver/pull/13667#discussion_r3993415163


##########
doc/admin-guide/files/records.yaml.en.rst:
##########
@@ -2433,6 +2433,29 @@ Security
    post body larger than this limit the response will be terminated with
    413 - Request Entity Too Large and logged accordingly.
 
+.. ts:cv:: CONFIG proxy.config.http.log_server_tcp_info INT 0
+   :reloadable:
+
+   Enables sampling of ``TCP_INFO`` on the origin connection, so that the round
+   trip time to the origin can be logged.

Review Comment:
   Normally once per transaction: one `getsockopt(TCP_INFO)` call when the 
origin response header is successfully parsed, provided the config and logging 
guards allow it. Each additional successfully parsed response header (for 
example, during a retry or redirect) can cause another read; a direct cache hit 
does not.
   
   The header-time snapshot keeps RTT close to origin TTFB. The kernel 
maintains these statistics independently, including across reused connections. 
I've made the frequency explicit in the docs. Collection is independent of the 
later `proxy.config.log.sampling_frequency` decision.
   



-- 
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.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to