Merge lp:~akopytov/percona-server/bug845019-5.1 into lp:percona-server/5.1

Proposed by Alexey Kopytov
Status: Merged
Merged at revision: 281
Proposed branch: lp:~akopytov/percona-server/bug845019-5.1
Merge into: lp:percona-server/5.1
Diff against target: 18 lines (+3/-1)
1 file modified
patches/response_time_distribution.patch (+3/-1)
To merge this branch: bzr merge lp:~akopytov/percona-server/bug845019-5.1
Reviewer Review Type Date Requested Status
Oleg Tsarev (community) Approve
Review via email: mp+75711@code.launchpad.net
To post a comment you must log in.
Revision history for this message
Alexey Kopytov (akopytov) wrote :
Revision history for this message
Oleg Tsarev (tsarev) wrote :

LGTM

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'patches/response_time_distribution.patch'
2--- patches/response_time_distribution.patch 2011-07-30 23:07:03 +0000
3+++ patches/response_time_distribution.patch 2011-09-16 11:15:29 +0000
4@@ -3119,11 +3119,13 @@
5
6 /**
7 @defgroup Runtime_Environment Runtime Environment
8-@@ -1777,6 +1778,7 @@
9+@@ -1777,6 +1778,9 @@
10
11 ulonglong end_utime_of_query= thd->current_utime();
12 ulonglong query_exec_time= get_query_exec_time(thd, end_utime_of_query);
13++#ifdef HAVE_RESPONSE_TIME_DISTRIBUTION
14 + query_response_time_collect(query_exec_time);
15++#endif
16
17 /*
18 Low long_query_time value most likely means user is debugging stuff and even

Subscribers

People subscribed via source and target branches