Merge lp:~laurynas-biveinis/percona-server/bug1214735-5.5 into lp:percona-server/5.5

Proposed by Laurynas Biveinis
Status: Merged
Approved by: Laurynas Biveinis
Approved revision: no longer in the source branch.
Merged at revision: 563
Proposed branch: lp:~laurynas-biveinis/percona-server/bug1214735-5.5
Merge into: lp:percona-server/5.5
Diff against target: 12 lines (+1/-1)
1 file modified
Percona-Server/storage/innobase/os/os0file.c (+1/-1)
To merge this branch: bzr merge lp:~laurynas-biveinis/percona-server/bug1214735-5.5
Reviewer Review Type Date Requested Status
Laurynas Biveinis (community) Approve
Review via email: mp+181338@code.launchpad.net

Description of the change

Fix bug 1214735 (os_file_set_atomic_writes() return value check is
inverted). The fix is trivial.

http://jenkins.percona.com/job/percona-server-5.5-param/827/

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

Trivial, self-approving.

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'Percona-Server/storage/innobase/os/os0file.c'
2--- Percona-Server/storage/innobase/os/os0file.c 2013-06-27 15:35:20 +0000
3+++ Percona-Server/storage/innobase/os/os0file.c 2013-08-21 15:46:46 +0000
4@@ -1677,7 +1677,7 @@
5 #endif /* USE_FILE_LOCK */
6
7 if (srv_use_atomic_writes && type == OS_DATA_FILE
8- && os_file_set_atomic_writes(name, file)) {
9+ && !os_file_set_atomic_writes(name, file)) {
10
11 *success = FALSE;
12 close(file);

Subscribers

People subscribed via source and target branches