Merge lp:~percona-dev/percona-server/release-5.1.53-12-slow_extended-fix into lp:~percona-dev/percona-server/release-5.1.53
Status: | Rejected | ||||||||
---|---|---|---|---|---|---|---|---|---|
Rejected by: | Alexey Kopytov | ||||||||
Proposed branch: | lp:~percona-dev/percona-server/release-5.1.53-12-slow_extended-fix | ||||||||
Merge into: | lp:~percona-dev/percona-server/release-5.1.53 | ||||||||
Diff against target: |
547 lines (+126/-28) 32 files modified
control_online_alter_index.patch (+1/-1) microsec_process.patch (+1/-1) mysql-test/slow_extended.patch/percona_slow_extended-combined-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-combined.result (+18/-0) mysql-test/slow_extended.patch/percona_slow_extended-combined.test (+6/-0) mysql-test/slow_extended.patch/percona_slow_extended-combined2-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-combined2.result (+12/-0) mysql-test/slow_extended.patch/percona_slow_extended-combined2.test (+4/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_sp_statements-cl-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_sp_statements-cl.result (+3/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_sp_statements-cl.test (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_timestamp_every-cl-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_timestamp_every-cl.result (+3/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_timestamp_every-cl.test (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_verbosity-cl-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_verbosity-cl.result (+9/-0) mysql-test/slow_extended.patch/percona_slow_extended-log_slow_verbosity-cl.test (+3/-0) mysql-test/slow_extended.patch/percona_slow_extended-slave_statements-and-use_global_long_query_time.test (+2/-2) mysql-test/slow_extended.patch/percona_slow_extended-slave_statements.test (+2/-2) mysql-test/slow_extended.patch/percona_slow_extended-slow_query_log_microseconds_timestamp-cl-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-slow_query_log_microseconds_timestamp-cl.result (+3/-0) mysql-test/slow_extended.patch/percona_slow_extended-slow_query_log_microseconds_timestamp-cl.test (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-use_global_long_query_time-cl-master.opt (+1/-0) mysql-test/slow_extended.patch/percona_slow_extended-use_global_long_query_time-cl.result (+3/-0) mysql-test/slow_extended.patch/percona_slow_extended-use_global_long_query_time-cl.test (+1/-0) optimizer_fix.patch (+1/-1) profiling_slow.patch (+11/-2) query_cache_enhance.patch (+6/-6) response-time-distribution.patch (+2/-2) slow_extended.patch (+23/-9) suppress_log_warning_1592.patch (+1/-1) userstat.patch (+1/-1) |
||||||||
To merge this branch: | bzr merge lp:~percona-dev/percona-server/release-5.1.53-12-slow_extended-fix | ||||||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Alexey Kopytov (community) | Disapprove | ||
Review via email: mp+44089@code.launchpad.net |
This proposal supersedes a proposal from 2010-12-17.
Description of the change
1) Improve tests for slow_extended
2) Update infrastructure scripts
3) Remove unnecessary test for show_slave_
4) fix bug #691234
5) fix bug #688643
6) Fixed tests for slow_extended.patch - replace "cat ./var/..." to "cat $MYSQLTEST_
-------
bool variables in mysql can be used without argument in command-line and config.
Example:
--use_global_
Variables what can be used without argument named "optional".
Bug #691234 provocated by "required" type of variable - as result,
--use_global_
interpreted as:
use_global_
as effect:
use_global_
-------
Command-line options update different from update system_variables.
Bug #668643 provocated by different update way for use_global_
I fixed this thought share the update to command-line option
Unmerged revisions
- 178. By Oleg Tsarev
-
bzr merge lp:~percona-dev/percona-server/release-5.1.53-slow_extended
- 177. By Oleg Tsarev
-
fix new line carrets
- 176. By Oleg Tsarev
-
fix new line carrets
- 175. By Oleg Tsarev
-
1) Improve tests for slow_extended
2) Update infrastructure scripts
3) Remove unnecessary test for show_slave_status_ nolock. patch
4) fix bug #691234
5) fix bug #688643 - 174. By kinoyasu <kinoyasu@gauntlet4>
-
backport adaptive_
checkpoint= keep_average and innodb_ log_block_ size from XtraDB on 5.5.7 dev-branch - 173. By Aleksandr Kuzminsky
-
fixed compile error of query_response_
time.cc from response_ time_distributi on.patch (bad offsets in patch) - 172. By kinoyasu <kinoyasu@gauntlet4>
-
fix bug690144
- 171. By Oleg Tsarev
-
add option --without-
response_ time_distributi on to response- time-distributi on.patch
Also added variable have_response_time_distributi on - 170. By Oleg Tsarev
-
fix bug 689830
- 169. By kinoyasu <kinoyasu@gauntlet4>
-
lp:~percona-dev/percona-server/release-5.1.51-12-expand_import_fix4autoinc is merged... (apply fix for innodb_
expand_ import. patch from Vamsi@FaceBook about autoinc)
Oleg, please don't be lazy and create separate MPs for every change or bugfix you would like to push.
When reviewing a MP, I would like to see the following:
1) Description of the problem being fixed by the patch I'm looking at.
2) Description of the way the patch addresses the problem (if it's not obvious)
3) The patch itself.
What I see instead is a bunch of changesets like this:
--- cut ---
178. By Oleg Tsarev on 2010-12-17
bzr merge lp:~percona-dev/percona-server/release-5.1.53-slow_extended
177. By Oleg Tsarev on 2010-12-17
fix new line carrets
176. By Oleg Tsarev on 2010-12-17
fix new line carrets
175. By Oleg Tsarev on 2010-12-17
1) Improve tests for slow_extended status_ nolock. patch
2) Update infrastructure scripts
3) Remove unnecessary test for show_slave_
4) fix bug #691234
5) fix bug #688643
--- cut ---
Then there is a bunch of changes in both code and test files. So I have to figure out which change correspond to which changeset *and* item from that changeset ("is this supposed to fix bug #691234 or bug #677643?"). Then I have to figure out the actual problem, because, believe it or not, a description like "Bug #668643 provocated by different update way for use_global_ long_query_ time" does NOT really describe anything. So I have to go read the bug report, figure out the problem, then look at your change again and make guesses about the proposed way to fix that problem.
This is _very_ time consuming. Please save me a few hours by just structuring your changes appropriately.