← Back to team overview

maria-developers team mailing list archive

Re: Review: MDEV-29075 Changing explicit_defaults_for_timestamp within stored procedure works inconsistently

 

Hi!

On Fri, 29 Jul 2022, 07:43 Alexander Barkov, <bar@xxxxxxxxxxx> wrote:

>    Hello Sergei,
>
> I have a couple of suggestions:
>

<cut>
>

I suggest considering these ways:
>
>
> 1. Change "bool explicitly_nullable" to "bool explicit_nullability",
> which will be:
>
> - false if nothing was specified
> - true if NULL or NOT NULL was specified.
>
>
> 2. Or don't store NOT_NULL_FLAG in Column_definition::flags at all.
>
> Add a new Column_definition enum member with three values for:
> - not specified
> - explicit NULL
> - explicit NOT NULL
>

> This would require 4 or 8 bytes extra per field (and each field definition
> is stored twice per table), so this is probably not a good idea.


Regards,
Monty

References