Merge lp:~tsarev/percona-server/5.1_fix_bug_838725_base into lp:percona-server/5.1

Proposed by Oleg Tsarev
Status: Merged
Approved by: Stewart Smith
Approved revision: no longer in the source branch.
Merged at revision: 322
Proposed branch: lp:~tsarev/percona-server/5.1_fix_bug_838725_base
Merge into: lp:percona-server/5.1
Diff against target: 34 lines (+11/-8)
2 files modified
patches/innodb_bug47167_test_fix.patch (+0/-8)
patches/innodb_swap_builtin_plugin.patch (+11/-0)
To merge this branch: bzr merge lp:~tsarev/percona-server/5.1_fix_bug_838725_base
Reviewer Review Type Date Requested Status
Stewart Smith (community) Approve
Laurynas Biveinis (community) Approve
Review via email: mp+78683@code.launchpad.net

Description of the change

NOTE: This branch NOT FOR MERGE, JUST FOR REVIEW

FAILED TESTS:
1. main.endspace main.index_merge_innodb main.rowid_order_innodb main.type_bit_innodb
https://code.launchpad.net/~tsarev/percona-server/5.1_fix_bug_832528/+merge/78687

2. main.percona_innodb_kill_idle_trx main.percona_innodb_kill_idle_trx_locks
https://code.launchpad.net/~tsarev/percona-server/fix_for_kill_idle_trx_test/+merge/78691

3. main.percona_server_variables_debug main.percona_server_variables_release
https://code.launchpad.net/~tsarev/percona-server/fix_for_kill_idle_trx_test/+merge/78691
https://code.launchpad.net/~tsarev/percona-server/5.1_fix_bug_860416/+merge/78692

FAILED TESTCASES (JUST DEBUG):
1. rpl.rpl_binlog_errors
https://code.launchpad.net/~tsarev/percona-server/5.1_fix_bug_794790/+merge/78694

DESCRIPTION:
1. Fixed bug #838725 - some percona's tests doesn't run after commit number 266 (patch patches/innodb_bug47167_test_fix.patch)
2. Remove innodb suite from default suites list (mysql-test-run.pl)

To post a comment you must log in.
Revision history for this message
Laurynas Biveinis (laurynas-biveinis) wrote :

LGTM

review: Approve
Revision history for this message
Stewart Smith (stewart) :
review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'patches/innodb_bug47167_test_fix.patch'
2--- patches/innodb_bug47167_test_fix.patch 2011-08-01 17:56:34 +0000
3+++ patches/innodb_bug47167_test_fix.patch 2011-10-08 00:04:25 +0000
4@@ -42,11 +42,3 @@
5
6 # Following are negative tests, all should fail.
7 --disable_warnings
8---- a/mysql-test/include/have_innodb.inc
9-+++ b/mysql-test/include/have_innodb.inc
10-@@ -1,4 +1,4 @@
11--if (`SELECT COUNT(*) = 0 FROM INFORMATION_SCHEMA.ENGINES WHERE engine = 'innodb' AND support IN ('YES', 'DEFAULT', 'ENABLED')`)
12-+if (`select count(*)>0 from information_schema.plugins where plugin_name like 'xtradb%'`)
13- {
14- --skip Test requires InnoDB.
15- }
16
17=== modified file 'patches/innodb_swap_builtin_plugin.patch'
18--- patches/innodb_swap_builtin_plugin.patch 2011-08-09 13:35:34 +0000
19+++ patches/innodb_swap_builtin_plugin.patch 2011-10-08 00:04:25 +0000
20@@ -198,3 +198,14 @@
21 MYSQL_PLUGIN_DYNAMIC(innodb_plugin, [ha_innodb_plugin.la])
22 MYSQL_PLUGIN_ACTIONS(innodb_plugin, [
23 AC_CHECK_HEADERS(sched.h)
24+--- a/mysql-test/mysql-test-run.pl
25++++ b/mysql-test/mysql-test-run.pl
26+@@ -157,7 +157,7 @@
27+ # executables will be used by the test suite.
28+ our $opt_vs_config = $ENV{'MTR_VS_CONFIG'};
29+
30+-my $DEFAULT_SUITES= "main,binlog,federated,rpl,rpl_ndb,ndb,innodb,innodb_plugin";
31++my $DEFAULT_SUITES= "main,binlog,federated,rpl,rpl_ndb,ndb,innodb_plugin";
32+ my $opt_suites;
33+
34+ our $opt_verbose= 0; # Verbose output, enable with --verbose

Subscribers

People subscribed via source and target branches