On Fri, 8 Jul 2022 at 00:34, Sergei Golubchik <serg@mariadb.org> wrote:
I'd suggest to add method, like

 struct Rpl_table_data
 {
   ...
   bool is_online_alter() { return copy_fields != NULL; }
   ...
 };

btw, it fits very well in your Rpl_table_data simplification.

> +                               | MODE_NO_AUTO_VALUE_ON_ZERO;

Right, and besides It reminds me a similar method from instant alter table.

--
Yours truly,
Nikita Malyavin