[tarantool-patches] Re: [PATCH v1 1/4] box: rename space->opts checks to checks_ast
Konstantin Osipov
kostja at tarantool.org
Fri Jan 18 21:00:52 MSK 2019
* Kirill Shcherbatov <kshcherbatov at tarantool.org> [19/01/10 23:12]:
> The checks field of space_opts object renamed to checks_ast to avoid a
> confusion due to introduction of a new object sql_check, representing
> precompiled VDBE program to execute.
Please use name check_constraint_expr
A common abbreviation for foreign key constraint is fk, check
constarint - ck, not null - nn, so an alternative family of names
would be fk_constraint, ck_constraint, nn_constraint. Feel free to
use it if you like a bit shorter form.
> + if (def->opts.checks_ast != NULL &&
> + sql_checks_resolve_space_def_reference(def->opts.checks_ast,
Please name the module 'check_constraint' or ck_constraint, and rename all the
functions accordingly.
--
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