[tarantool-patches] Re: [PATCH 1/6] sql: move constraint name to struct contraint_parse

Konstantin Osipov kostja at tarantool.org
Thu Jan 17 13:56:13 MSK 2019


* n.pettik <korablev at tarantool.org> [19/01/16 23:09]:
>  struct constraint_parse {
>         /** Name of table which constraint belongs to. */
> -       struct SrcList *table_name;
> +       struct SrcList *table;
>         /** Name of the constraint currently being parsed. */
>         struct Token name;
> +       /**
> +        * List of columns involved in constraint definition:
> +        * indexed or referencing columns.
> +        */
> +       struct ExprList *cols;

Why not use constraint_def here and all future parser-specific
objects? _def is our widely used abbreviation for "definition". A
parser creates a constraint definition, and then it is used to
create a runtime constraint object - struct constraint.


-- 
Konstantin Osipov, Moscow, Russia, +7 903 626 22 32
http://tarantool.io - www.twitter.com/kostja_osipov




More information about the Tarantool-patches mailing list