Prettier wfLogProfilingData() log message
authorBryan Davis <bd808@wikimedia.org>
Mon, 17 Nov 2014 17:32:45 +0000 (10:32 -0700)
committerBryan Davis <bd808@wikimedia.org>
Mon, 17 Nov 2014 17:41:49 +0000 (10:41 -0700)
Make the profiler logging output introduced in Iae11e1e a little nicer.
Stop adding the Profiler::getRawData() information and add the URL and
textual report to the default log message. When testing with a Monolog
logger and Logstash the getRawData output from the Xhprof profiler was
found to be too big to be usable. The default message output was found
to be less informative than desirable when a PSR-3 logger other than
MWLoggerLegacyLogger was used. MWLoggerLegacyLogger implements special
backwards compatible formatting for the profileoutput channel that will
not be effected by this change.

Change-Id: Id326aed4f72a3489fac30661c101ac3bb3af2530

includes/GlobalFunctions.php

index b499904..71c3791 100644 (file)
@@ -1226,10 +1226,9 @@ function wfLogProfilingData() {
        }
 
        $ctx['output'] = $profiler->getOutput();
-       $ctx['profile'] = $profiler->getRawData();
 
        $log = MWLogger::getInstance( 'profileoutput' );
-       $log->info( 'Elapsed: {elapsed}', $ctx );
+       $log->info( "Elapsed: {elapsed}; URL: <{url}>\n{output}", $ctx );
 }
 
 /**