Misc[MQB]: remove log Stats dumped; log admin cmd execution time (#983)

Signed-off-by: Evgeny Malygin <[email protected]>

Author: 678098

src/groups/mqb/mqba/mqba_application.cpp

@@ -46,6 +46,7 @@
 
 #include <bmqscm_version.h>
 #include <bmqst_statcontext.h>
+#include <bmqsys_operationlogger.h>
 #include <bmqsys_time.h>
 #include <bmqu_memoutstream.h>
 
@@ -977,11 +978,18 @@ int Application::processCommandCb(
     const bsl::function<void(int, const bsl::string&)>& onProcessedCb,
     bool                                                fromReroute)
 {
-    bmqu::MemOutStream os;
-    int                rc = processCommand(source, cmd, os, fromReroute);
+    bmqsys::OperationLogger opLogger(d_allocator_p);
+    // Set operation name later when we have a return code
+    opLogger.start();
+
+    bmqu::MemOutStream os(d_allocator_p);
+    const int          rc = processCommand(source, cmd, os, fromReroute);
+    opLogger.operation() << "Process command '" << cmd << "' (rc = " << rc
+                         << ")";
 
     onProcessedCb(rc, os.str());
 
+    // `opLogger` logs execution time on destruction
     return rc;  // RETURN
 }
 

src/groups/mqb/mqbstat/mqbstat_printer.cpp

@@ -271,10 +271,6 @@ void Printer::logStats()
 {
     ++d_lastStatId;
 
-    // Put a 'reference' in the main log file. We do that first in case it
-    // crashes/hangs in dump stat, this will help figuring it
-    BALL_LOG_INFO << "Stats dumped [id: " << d_lastStatId << "]";
-
     // Dump to statslog file
     // Prepare the log record and associated attributes
     ball::Record            record;