[wp-trac] [WordPress Trac] #14859: WP-DB Add query_done Action for Performance Logging Plugins

WordPress Trac wp-trac at lists.automattic.com
Tue Sep 14 01:00:34 UTC 2010


#14859: WP-DB Add query_done Action for Performance Logging Plugins
-------------------------+--------------------------------------------------
 Reporter:  sexytyranno  |       Owner:                 
     Type:  enhancement  |      Status:  new            
 Priority:  normal       |   Milestone:  Awaiting Review
Component:  Database     |     Version:  3.0.1          
 Severity:  normal       |    Keywords:                 
-------------------------+--------------------------------------------------

Comment(by sexytyranno):

 The newly create action will only actually work when SAVEQUERIES is
 define()d, so on most production sites using sane plug-ins, shouldn't
 cause any performance impact.

 Really, I've not ever seen SAVEQUERIES defined before, except in the
 context of debugging issues, so the impact should be pretty narrowed.

 If the core WP group would prefer, we could also add another low-publicity
 define() into wp-config.php to enable the query_done action, so as to make
 it one of those things where a site administrator would have manually and
 knowingly enable it.

 To this extent, perhaps we can insert a warning near the top of wp-admin
 somewhere outlining that:

 - It's enabled. It's typically defined in wp-config.php, but it may be
 defined in a plug-in or theme. (Silly and bad plug-in/theme author!)

 - Debugging only feature. Should not be used on live sites.

 - Possible performance impact while it's enabled.

 Some of the above may be ported to a "What does this mean?" link, rather
 than having a really large warning box.

-- 
Ticket URL: <http://core.trac.wordpress.org/ticket/14859#comment:2>
WordPress Trac <http://core.trac.wordpress.org/>
WordPress blogging software


More information about the wp-trac mailing list