bneradt commented on a change in pull request #6931:
URL: https://github.com/apache/trafficserver/pull/6931#discussion_r444480041
##########
File path: src/traffic_server/traffic_server.cc
##########
@@ -2094,7 +2116,11 @@ main(int /* argc ATS_UNUSED */, const char **argv)
if (delay_p && ink_atomic_cas(&delay_listen_for_cache_p, 0, 1)) {
Debug("http_listen", "Delaying listen, waiting for cache
initialization");
} else {
+ // Since the previous delay_listen_for_cache_p was not 0, then the
+ // cache must have been initialized already.
Review comment:
Alan pointed out offline that we should take delay_p, tied to the value
of wait_for_cache, into consideration with this comment. I've updated the
comment accordingly.
Thanks for the review Alan!
----------------------------------------------------------------
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]
