Code review comment for lp:~vlad-lesin/percona-server/5.6-query_response_time-rw-commands

Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

As discussed yesterday on IRC:

- THD::update_server_status uses utime_after_query too
do you know whether updating the field might cause the change of behavior somewhere else?
-
64 - query_response_time_collect(thd->utime_after_query - thd->utime_after_lock);
65 + query_response_time_collect(query_type, thd->utime_after_query - thd->start_utime);
before: utime after lock
after: start utime
why?

review: Needs Information

« Back to merge proposal