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] SQL types changes
Date: Thu, 7 Mar 2019 16:14:00 +0300 [thread overview]
Message-ID: <cover.1551964360.git.korablev@tarantool.org> (raw)
Branch: https://github.com/tarantool/tarantool/tree/np/gh-4019-types-cleanup
Issue:
https://github.com/tarantool/tarantool/issues/4019
https://github.com/tarantool/tarantool/issues/4023
This patch-set polishes current situation with SQL types. Several type
aliases (CHAR/DECIMAL/NUMERIC) were removed; TIME-like types were
completely removed. BLOB was replaced with SCALAR to avoid any
confusions in using that type. For more details, see content of patches
and document request in the last patch.
Nikita Pettik (4):
sql: remove support of DATE/TIME from parser
sql: remove support of CHAR type from parser
sql: remove support of NUMERIC type from parser
sql: replace BLOB as column type with SCALAR
extra/mkkeywordhash.c | 18 +++----
src/box/sql/date.c | 9 ++--
src/box/sql/parse.y | 79 +++++++++++++++-------------
src/box/sql/vdbe.c | 2 -
src/box/sql/vdbemem.c | 14 +----
test/sql-tap/analyze9.test.lua | 2 +-
test/sql-tap/autoinc.test.lua | 2 +-
test/sql-tap/blob.test.lua | 4 +-
test/sql-tap/boundary1.test.lua | 2 +-
test/sql-tap/boundary3.test.lua | 2 +-
test/sql-tap/cast.test.lua | 60 ++++++++++-----------
test/sql-tap/collation.test.lua | 2 +-
test/sql-tap/date.test.lua | 9 ++--
test/sql-tap/default.test.lua | 4 +-
test/sql-tap/delete4.test.lua | 2 +-
test/sql-tap/distinct.test.lua | 11 ++--
test/sql-tap/e_expr.test.lua | 14 +++--
test/sql-tap/eqp.test.lua | 4 +-
test/sql-tap/fkey1.test.lua | 4 +-
test/sql-tap/in1.test.lua | 2 +-
test/sql-tap/in3.test.lua | 9 ++--
test/sql-tap/in4.test.lua | 2 +-
test/sql-tap/index1.test.lua | 2 +-
test/sql-tap/index4.test.lua | 4 +-
test/sql-tap/resolver01.test.lua | 4 +-
test/sql-tap/select6.test.lua | 6 +--
test/sql-tap/substr.test.lua | 2 +-
test/sql-tap/suite.ini | 4 ++
test/sql-tap/table.test.lua | 30 ++++++-----
test/sql-tap/tkt-7bbfb7d442.test.lua | 4 +-
test/sql-tap/tkt-bd484a090c.test.lua | 9 +++-
test/sql-tap/tkt1443.test.lua | 2 +-
test/sql-tap/tkt1444.test.lua | 2 +-
test/sql-tap/tkt1449.test.lua | 14 ++---
test/sql-tap/tkt2192.test.lua | 8 ++-
test/sql-tap/tkt3791.test.lua | 10 ++--
test/sql-tap/types2.test.lua | 8 +--
test/sql-tap/where2.test.lua | 4 +-
test/sql-tap/whereB.test.lua | 22 ++++----
test/sql/collation.result | 10 ++--
test/sql/collation.test.lua | 10 ++--
test/sql/drop-index.result | 2 +-
test/sql/drop-index.test.lua | 2 +-
test/sql/errinj.result | 2 +-
test/sql/errinj.test.lua | 2 +-
test/sql/gh-2929-primary-key.result | 6 +--
test/sql/gh-2929-primary-key.test.lua | 6 +--
test/sql/gh-3888-values-blob-assert.result | 8 +--
test/sql/gh-3888-values-blob-assert.test.lua | 4 +-
test/sql/gh2251-multiple-update.result | 2 +-
test/sql/gh2251-multiple-update.test.lua | 2 +-
test/sql/persistency.result | 2 +-
test/sql/persistency.test.lua | 2 +-
test/sql/row-count.result | 4 +-
test/sql/row-count.test.lua | 4 +-
test/sql/transition.result | 4 +-
test/sql/transition.test.lua | 4 +-
test/sql/triggers.result | 4 +-
test/sql/triggers.test.lua | 4 +-
test/sql/types.result | 10 ++--
test/sql/types.test.lua | 7 ++-
61 files changed, 249 insertions(+), 244 deletions(-)
--
2.15.1
next reply other threads:[~2019-03-07 13:14 UTC|newest]
Thread overview: 11+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-03-07 13:14 Nikita Pettik [this message]
2019-03-07 13:14 ` [tarantool-patches] [PATCH 1/4] sql: remove support of DATE/TIME from parser Nikita Pettik
2019-03-07 13:34 ` [tarantool-patches] " Konstantin Osipov
2019-03-07 13:14 ` [tarantool-patches] [PATCH 2/4] sql: remove support of CHAR type " Nikita Pettik
2019-03-07 13:35 ` [tarantool-patches] " Konstantin Osipov
2019-03-07 13:14 ` [tarantool-patches] [PATCH 3/4] sql: remove support of NUMERIC " Nikita Pettik
2019-03-07 13:36 ` [tarantool-patches] " Konstantin Osipov
2019-03-07 13:14 ` [tarantool-patches] [PATCH 4/4] sql: replace BLOB as column type with SCALAR Nikita Pettik
2019-03-07 13:40 ` [tarantool-patches] " Konstantin Osipov
2019-03-07 14:00 ` n.pettik
2019-03-07 14:14 ` [tarantool-patches] Re: [PATCH 0/4] SQL types changes Kirill Yukhin
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.1551964360.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] SQL types changes' \
/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