[Tarantool-patches] [PATCH v2 0/3] Add ability to drop constraints
Roman Khabibov
roman.habibov at tarantool.org
Sat Feb 29 15:46:43 MSK 2020
@ChangeLog
- Add function space_index_by_name().
- Add ability to spacify searching index in OP_SDelete.
- Add function vdbe_emit_index_drop() to drop index by name.
- Remove box_index_by_name() from build.c and pragma.c.
- Rewrite function sql_drop_foreign_key() to sql_drop_constraint().
- Add ability to drop all the types of constraints.
- Modify "no such constrait" error message.
Hello, everybody!
As Nikita requested, I performed the patch to remove
box_index_by_name() from build.c and pragma.c, but what about
analyze.c? Still had to add space_index_by_name() to use in
pragma.c, beacuse struct index is required.
Roman Khabibov (3):
sql: improve "no such constraint" error message
sql: don't select from _index during parsing
sql: support constraint drop
src/box/constraint_id.h | 1 +
src/box/errcode.h | 2 +-
src/box/space.h | 15 ++++
src/box/sql/alter.c | 2 +-
src/box/sql/build.c | 143 ++++++++++++++++++++++-------------
src/box/sql/parse.y | 4 +-
src/box/sql/parse_def.h | 11 +--
src/box/sql/pragma.c | 8 +-
src/box/sql/sqlInt.h | 7 +-
src/box/sql/vdbe.c | 10 ++-
test/sql-tap/alter2.test.lua | 2 +-
test/sql/checks.result | 2 +-
test/sql/constraint.result | 81 ++++++++++++++++++++
test/sql/constraint.test.lua | 28 +++++++
14 files changed, 240 insertions(+), 76 deletions(-)
--
2.21.0 (Apple Git-122)
More information about the Tarantool-patches
mailing list