Merge lp:~dshrews/drizzle/bug726741_block_flush_analyze into lp:drizzle/7.0

Proposed by David Shrewsbury
Status: Merged
Approved by: Brian Aker
Approved revision: 2208
Merged at revision: 2212
Proposed branch: lp:~dshrews/drizzle/bug726741_block_flush_analyze
Merge into: lp:drizzle/7.0
Diff against target: 29 lines (+0/-9)
2 files modified
drizzled/statement/analyze.cc (+0/-2)
drizzled/statement/flush.cc (+0/-7)
To merge this branch: bzr merge lp:~dshrews/drizzle/bug726741_block_flush_analyze
Reviewer Review Type Date Requested Status
Drizzle Developers Pending
Review via email: mp+51622@code.launchpad.net

Description of the change

Do not pass FLUSH or ANALYZE statements through the replication stream.

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
1=== modified file 'drizzled/statement/analyze.cc'
2--- drizzled/statement/analyze.cc 2011-02-21 18:33:47 +0000
3+++ drizzled/statement/analyze.cc 2011-02-28 20:21:25 +0000
4@@ -34,8 +34,6 @@
5 assert(first_table == all_tables && first_table != 0);
6 Select_Lex *select_lex= &getSession()->getLex()->select_lex;
7 bool res= analyze_table(getSession(), first_table, &check_opt);
8- /* ! we write after unlocking the table */
9- write_bin_log(getSession(), *getSession()->getQueryString());
10 select_lex->table_list.first= (unsigned char*) first_table;
11 getSession()->getLex()->query_tables= all_tables;
12
13
14=== modified file 'drizzled/statement/flush.cc'
15--- drizzled/statement/flush.cc 2011-02-21 18:33:47 +0000
16+++ drizzled/statement/flush.cc 2011-02-28 20:21:25 +0000
17@@ -39,13 +39,6 @@
18 */
19 if (not reloadCache())
20 {
21- /*
22- * We WANT to write and we CAN write.
23- * ! we write after unlocking the table.
24- *
25- * Presumably, RESET and binlog writing doesn't require synchronization
26- */
27- write_bin_log(getSession(), *getSession()->getQueryString());
28 getSession()->my_ok();
29 }
30

Subscribers

People subscribed via source and target branches