Merge lp:~greenlion/my-movies/my-movies into lp:my-movies

Proposed by Swany
Status: Needs review
Proposed branch: lp:~greenlion/my-movies/my-movies
Merge into: lp:my-movies
Diff against target: 21 lines (+3/-3)
1 file modified
lib/instrumentation.php (+3/-3)
To merge this branch: bzr merge lp:~greenlion/my-movies/my-movies
Reviewer Review Type Date Requested Status
Swany (community) Approve
Review via email: mp+113131@code.launchpad.net

Description of the change

Fixed a problem where SQL comments are not added to SQL statements.

To post a comment you must log in.
Revision history for this message
Swany (greenlion) :
review: Approve

Unmerged revisions

6. By Swany

Fixed a problem where the SQL being emitted by the application was not properly embedded with comments.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'lib/instrumentation.php'
--- lib/instrumentation.php 2010-09-21 21:45:42 +0000
+++ lib/instrumentation.php 2012-07-02 23:12:21 +0000
@@ -534,8 +534,8 @@
534534
535 535
536 while($retry_count < MySQL_perf::$deadlock_try_limit) {536 while($retry_count < MySQL_perf::$deadlock_try_limit) {
537 $instance->increment('mysql_query_count', 1);537 $instance->increment('mysql_query_count', 1);
538 $instance->instrument_query($sql, $retry_count);538 $sql = $instance->instrument_query($sql, $retry_count);
539 539
540 $result = false;540 $result = false;
541 $errno = -1;541 $errno = -1;
@@ -564,4 +564,4 @@
564 } 564 }
565}565}
566566
567?>
568\ No newline at end of file567\ No newline at end of file
568?>

Subscribers

People subscribed via source and target branches

to all changes: