bneradt commented on a change in pull request #6806:
URL: https://github.com/apache/trafficserver/pull/6806#discussion_r486616994



##########
File path: proxy/logging/LogObject.cc
##########
@@ -1241,6 +1241,22 @@ LogObjectManager::roll_files(long time_now)
   return num_rolled;
 }
 
+void
+LogObjectManager::reopen_moved_log_files()
+{
+  for (auto &_object : this->_objects) {
+    _object->m_logFile->reopen_if_moved();
+  }
+
+  ACQUIRE_API_MUTEX("A LogObjectManager::reopen_moved_log_files");

Review comment:
       I believe @randall is correct. I copied this convention from the other 
examples of this in this file (such as LogObjectManager::display).




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


Reply via email to