maria-developers team mailing list archive

[Sergei Golubchik <serg@xxxxxxxxxxx>]

Hi, Alexander!

On Jul 20, Alexander Barkov wrote:
> >> In addition to the original MySQL changes in mtr, I also added
> >> two tests that describe specific behavior:
> >> sys_vars.explicit_defaults_for_timestamp_off
> >> sys_vars.explicit_defaults_for_timestamp_on
> >
> > If they're functional tests, their file names typically end with _func.
> > Although it's not that important now, when all_vars tests is disabled.
> 
> Should I rename them to:
> 
> sys_vars.explicit_defaults_for_timestamp_func_off
> sys_vars.explicit_defaults_for_timestamp_func_on
> 
> or
> 
> sys_vars.explicit_defaults_for_timestamp_off_func
> sys_vars.explicit_defaults_for_timestamp_on_func
> 
> ?

If you'd prefer to rename - please check how other functional sys_vars
tests are named.

But I suspect the distinction between basic and functional tests will
start to disappear now, as we no longer create obligatory basic tests
for every new variable. So, it's ok not to rename.

> Okey to push after rename?

No, sorry, I haven't checked the patch yet, I only replied to this your
email.

Regards,
Sergei