diff options
author | Brion Vibber <brion@pobox.com> | 2010-01-21 11:07:52 -0800 |
---|---|---|
committer | Brion Vibber <brion@pobox.com> | 2010-01-21 11:07:52 -0800 |
commit | 8dafe09ab25335c26e33548e988828a72e984239 (patch) | |
tree | 3906e9896425f3bf7b95bd12215fc2cfcb31a243 /classes | |
parent | 1d64ba6602f7ccd4c049167355445ca3600c28e6 (diff) |
Option to log slow db queries or all db queries
$config['db']['log_queries'] = true; // all
$config['db']['log_slow_queries'] = 10; // queries taking > 10 seconds
Diffstat (limited to 'classes')
-rw-r--r-- | classes/Memcached_DataObject.php | 33 |
1 files changed, 33 insertions, 0 deletions
diff --git a/classes/Memcached_DataObject.php b/classes/Memcached_DataObject.php index 4ecab9db6..6ceb59e8b 100644 --- a/classes/Memcached_DataObject.php +++ b/classes/Memcached_DataObject.php @@ -315,6 +315,39 @@ class Memcached_DataObject extends DB_DataObject return new ArrayWrapper($cached); } + /** + * sends query to database - this is the private one that must work + * - internal functions use this rather than $this->query() + * + * Overridden to do logging. + * + * @param string $string + * @access private + * @return mixed none or PEAR_Error + */ + function _query($string) + { + $start = microtime(true); + $result = parent::_query($string); + $delta = microtime(true) - $start; + + $limit = common_config('db', 'log_slow_queries'); + if (($limit > 0 && $delta >= $limit) || common_config('db', 'log_queries')) { + $clean = $this->sanitizeQuery($string); + common_log(LOG_DEBUG, sprintf("DB query (%0.3fs): %s", $delta, $clean)); + } + return $result; + } + + // Sanitize a query for logging + // @fixme don't trim spaces in string literals + function sanitizeQuery($string) + { + $string = preg_replace('/\s+/', ' ', $string); + $string = trim($string); + return $string; + } + // We overload so that 'SET NAMES "utf8"' is called for // each connection |