Code review comment for lp:~mariosplivalo/charms/trusty/percona-cluster/enable-binlogs

Revision history for this message
Felipe Reyes (freyes) wrote :

Hi Mario,

This feature makes a lot of sense, and it worked for me, but please add unit tests for your patch, I also have minor comments in the diff.

Thanks,

review: Needs Fixing

« Back to merge proposal