Merge lp:~tsarev/percona-server/test55_830199 into lp:percona-server/5.5

Proposed by Oleg Tsarev
Status: Superseded
Proposed branch: lp:~tsarev/percona-server/test55_830199
Merge into: lp:percona-server/5.5
Diff against target: 107 lines (+52/-12)
1 file modified
patches/slow_extended.patch (+52/-12)
To merge this branch: bzr merge lp:~tsarev/percona-server/test55_830199
Reviewer Review Type Date Requested Status
Percona developers Pending
Review via email: mp+76386@code.launchpad.net

This proposal supersedes a proposal from 2011-09-21.

This proposal has been superseded by a proposal from 2011-09-21.

Description of the change

Fix bug #830199 - thd->last_errno was not updated on handle_condition (bug was introduced while porting slow_extended.patch 5.1 to 5.5)

To post a comment you must log in.
Revision history for this message
Oleg Tsarev (tsarev) wrote :

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'patches/slow_extended.patch'
2--- patches/slow_extended.patch 2011-09-21 08:35:45 +0000
3+++ patches/slow_extended.patch 2011-09-21 11:36:23 +0000
4@@ -549,15 +549,16 @@
5
6 /**
7 Dumps a text description of a thread, its security context
8-@@ -912,6 +943,7 @@
9+@@ -907,6 +938,8 @@
10+ const char* msg,
11+ MYSQL_ERROR ** cond_hdl)
12+ {
13++ last_errno= sql_errno;
14++
15+ if (!m_internal_handler)
16+ {
17 *cond_hdl= NULL;
18- return FALSE;
19- }
20-+ last_errno= sql_errno;
21-
22- for (Internal_error_handler *error_handler= m_internal_handler;
23- error_handler;
24-@@ -3656,6 +3688,12 @@
25+@@ -3656,6 +3689,12 @@
26 first_successful_insert_id_in_prev_stmt;
27 backup->first_successful_insert_id_in_cur_stmt=
28 first_successful_insert_id_in_cur_stmt;
29@@ -570,7 +571,7 @@
30
31 if ((!lex->requires_prelocking() || is_update_query(lex->sql_command)) &&
32 !is_current_stmt_binlog_format_row())
33-@@ -3676,6 +3714,14 @@
34+@@ -3676,6 +3715,14 @@
35 cuted_fields= 0;
36 transaction.savepoints= 0;
37 first_successful_insert_id_in_cur_stmt= 0;
38@@ -585,7 +586,7 @@
39 }
40
41
42-@@ -3738,6 +3784,12 @@
43+@@ -3738,6 +3785,12 @@
44 */
45 examined_row_count+= backup->examined_row_count;
46 cuted_fields+= backup->cuted_fields;
47@@ -1332,7 +1333,7 @@
48 };
49
50
51-@@ -276,9 +280,11 @@
52+@@ -276,10 +280,11 @@
53
54 /* SHOW PROFILES */
55 bool show_profiles();
56@@ -1383,7 +1384,6 @@
57 +EVAL SET GLOBAL slow_query_log_file="$log_file_full_path";
58 +SET GLOBAL slow_query_log=ON;
59 +--enable_query_log
60-\ No newline at end of file
61 --- /dev/null
62 +++ b/mysql-test/include/log_stop.inc
63 @@ -0,0 +1,7 @@
64@@ -2384,3 +2384,43 @@
65 +++ b/mysql-test/t/percona_log_slow_global_control_default.test
66 @@ -0,0 +1 @@
67 +SET GLOBAL slow_query_log_use_global_control=default;
68+--- /dev/null
69++++ b/mysql-test/r/percona_slow_extended_log_error.result
70+@@ -0,0 +1,11 @@
71++SET GLOBAL long_query_time=0;
72++SET GLOBAL slow_query_log_timestamp_always=ON;
73++SET long_query_time=0;
74++DROP TABLE IF EXISTS t1;
75++CREATE TABLE t1 (a INT);
76++[log_start.inc] percona.slow_extended.error_on_quit
77++CREATE TABLE t1 (a INT);
78++ERROR 42S01: Table 't1' already exists
79++[log_stop.inc] percona.slow_extended.error_on_quit
80++[log_grep.inc] file: percona.slow_extended.error_on_quit pattern: Last_errno: 1050
81++[log_grep.inc] lines: 1
82+--- /dev/null
83++++ b/mysql-test/t/percona_slow_extended_log_error.test
84+@@ -0,0 +1,23 @@
85++--let log_file=percona.slow_extended.error_on_quit
86++
87++SET GLOBAL long_query_time=0;
88++SET GLOBAL slow_query_log_timestamp_always=ON;
89++
90++connect (conn1,localhost,root,,);
91++connection conn1;
92++SET long_query_time=0;
93++--disable_warnings
94++DROP TABLE IF EXISTS t1;
95++--enable_warnings
96++CREATE TABLE t1 (a INT);
97++--source include/log_start.inc
98++--error ER_TABLE_EXISTS_ERROR
99++CREATE TABLE t1 (a INT);
100++disconnect conn1;
101++connection default;
102++--source include/log_stop.inc
103++--let grep_pattern = Last_errno: 1050
104++--source include/log_grep.inc
105++SET GLOBAL long_query_time=default;
106++SET GLOBAL slow_query_log_timestamp_always=default;
107++DROP TABLE t1;

Subscribers

People subscribed via source and target branches