From: Nikita Pettik <korablev@tarantool.org> To: tarantool-patches@freelists.org Cc: v.shpilevoy@tarantool.org, Nikita Pettik <korablev@tarantool.org> Subject: [tarantool-patches] [PATCH 0/4] Fixes in SQL involving no-pk or no-format spaces Date: Fri, 29 Mar 2019 21:24:20 +0300 [thread overview] Message-ID: <cover.1553883575.git.korablev@tarantool.org> (raw) Branch: https://github.com/tarantool/tarantool/tree/np/fix-no-index-no-format-DDL This patch-set consists of several (almost independent) fixes of bugs which can result in crashes. All of them are based on situation when SQL appeals to spaces which lack format or indexes. Nikita Pettik (4): Drop foreign keys before indexes in space:drop() Fix creation of FK constraint in case of no child's PK sql: disallow creation of index on space without format sql: disallow creation of FK referencing space without PK src/box/alter.cc | 2 +- src/box/lua/schema.lua | 6 +++--- src/box/sql/build.c | 23 +++++++++++++++++++++- src/box/sql/delete.c | 4 +--- src/box/sql/sqlInt.h | 12 ++++++++++++ test/sql-tap/lua-tables.test.lua | 17 ++++++++++++++++- test/sql/foreign-keys.result | 41 ++++++++++++++++++++++++++++++++++++++++ test/sql/foreign-keys.test.lua | 21 ++++++++++++++++++++ 8 files changed, 117 insertions(+), 9 deletions(-) -- 2.15.1
next reply other threads:[~2019-03-29 18:24 UTC|newest] Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-03-29 18:24 Nikita Pettik [this message] 2019-03-29 18:24 ` [tarantool-patches] [PATCH 1/4] Drop foreign keys before indexes in space:drop() Nikita Pettik 2019-03-29 18:24 ` [tarantool-patches] [PATCH 2/4] Fix creation of FK constraint in case of no child's PK Nikita Pettik 2019-04-01 13:41 ` [tarantool-patches] " Vladislav Shpilevoy 2019-03-29 18:24 ` [tarantool-patches] [PATCH 3/4] sql: disallow creation of index on space without format Nikita Pettik 2019-03-29 18:24 ` [tarantool-patches] [PATCH 4/4] sql: disallow creation of FK referencing space without PK Nikita Pettik
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=cover.1553883575.git.korablev@tarantool.org \ --to=korablev@tarantool.org \ --cc=tarantool-patches@freelists.org \ --cc=v.shpilevoy@tarantool.org \ --subject='Re: [tarantool-patches] [PATCH 0/4] Fixes in SQL involving no-pk or no-format spaces' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox