Hi Sachin! I'll review the patch. If Monty wants to chip in he can do so. 1. Indentation is wrong for the patch, you have an extra step of indentation. 2. I would use only one return true statement, after the if/else statement. 3. For singular subject sentence use does instead of do. So: 'Flashback does not support %s'. Same for the error message 'MariaDB Galera does not support binlog format: %s' 4. A testcase would be really useful for this. Vicențiu On Fri, 7 Apr 2017 at 09:46 Sachin Setiya <sachin.setiya@mariadb.com> wrote: Hi Monty!, Please review patch for 12017 and 12018 . this patch solves both the problem. -- Regards Sachin Setiya Software Engineer at MariaDB