Merge lp://staging/~percona-dev/percona-server/release-5.1.52-12-tsarev_2010_12_03-03-response_time_distribution-fix into lp://staging/~percona-dev/percona-server/release-5.1.52-12

Proposed by Oleg Tsarev
Status: Merged
Approved by: Vadim Tkachenko
Approved revision: no longer in the source branch.
Merged at revision: 152
Proposed branch: lp://staging/~percona-dev/percona-server/release-5.1.52-12-tsarev_2010_12_03-03-response_time_distribution-fix
Merge into: lp://staging/~percona-dev/percona-server/release-5.1.52-12
Prerequisite: lp://staging/~percona-dev/percona-server/release-5.1.52-12-tsarev_2010_12_03-02-enviroment_variable-fix
Diff against target: 305 lines (+57/-37)
8 files modified
mysql-test/response-time-distribution.patch/percona_query_response_time-replication.result (+14/-4)
mysql-test/response-time-distribution.patch/percona_query_response_time-replication.test (+8/-4)
mysql-test/response-time-distribution.patch/percona_query_response_time-stored.result (+7/-10)
mysql-test/response-time-distribution.patch/percona_query_response_time-stored.test (+4/-1)
mysql-test/response-time-distribution.patch/percona_query_response_time.result (+14/-17)
mysql-test/response-time-distribution.patch/percona_query_response_time.test (+3/-0)
mysql-test/show_slave_status_nolock.patch/percona_show_slave_status_nolock.test (+1/-0)
response-time-distribution.patch (+6/-1)
To merge this branch: bzr merge lp://staging/~percona-dev/percona-server/release-5.1.52-12-tsarev_2010_12_03-03-response_time_distribution-fix
Reviewer Review Type Date Requested Status
Percona developers Pending
Review via email: mp+42755@code.staging.launchpad.net

This proposal supersedes a proposal from 2010-12-04.

Description of the change

Fix test for response-time-distribution.
I add DBUG_EXECUTE_IF_VARIABLE "response_time_distribution_log_only_more_300_milliseconds".
If you enable this variable by 'SET SESSION debug="+d,response_time_distribution_log_only_more_300_milliseconds"'
all queries what work less than 300 milliseconds don't count in QUERY_RESPONSE_TIME.
As result - now tests work stable.

To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: