Code review comment for lp:~akopytov/percona-xtrabackup/read-server-options-with-show-variables

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

Hi Alexey,

The patch looks good to me. I agree that getting rid of innobackupex will solve many issues and I believe we should include https://blueprints.launchpad.net/percona-xtrabackup/+spec/rewrite-innobackupex-in-c in the roadmap if it is not there yet.

Approved.

review: Approve

« Back to merge proposal