Merge lp:~gl-az/percona-xtrabackup/bug1273196-2.2 into lp:percona-xtrabackup/2.2

Proposed by George Ormond Lorch III
Status: Merged
Approved by: Sergei Glushchenko
Approved revision: no longer in the source branch.
Merge reported by: Alexey Kopytov
Merged at revision: not available
Proposed branch: lp:~gl-az/percona-xtrabackup/bug1273196-2.2
Merge into: lp:percona-xtrabackup/2.2
To merge this branch: bzr merge lp:~gl-az/percona-xtrabackup/bug1273196-2.2
Reviewer Review Type Date Requested Status
Sergei Glushchenko (community) g2 Approve
Review via email: mp+206542@code.launchpad.net

Description of the change

bug 1273196 - Streaming issue with xbcrypt

xbcrypt read failures are due to partial reads happening on a stdin file because reads are not using either MY_FULL_IO nor looping and rereading on a partial (> 0) read.

To fix, simple re-read loops are added for each read point to progressively continue to read into the buffer as long as something was read.

jenkins http://jenkins.percona.com/view/PXB%202.2/job/percona-xtrabackup-2.2-param/102/

To post a comment you must log in.
Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

See comments for 2.1

review: Needs Fixing (g2)
Revision history for this message
George Ormond Lorch III (gl-az) wrote :

Same as 2.1, cood catch and simple fix, new jenkins: http://jenkins.percona.com/view/PXB%202.2/job/percona-xtrabackup-2.2-param/106/

Revision history for this message
Sergei Glushchenko (sergei.glushchenko) wrote :

Approve

review: Approve (g2)

Subscribers

People subscribed via source and target branches