Tarantool development patches archive
 help / color / mirror / Atom feed
* [tarantool-patches] [PATCH 0/4] SQL types changes
@ 2019-03-07 13:14 Nikita Pettik
  2019-03-07 13:14 ` [tarantool-patches] [PATCH 1/4] sql: remove support of DATE/TIME from parser Nikita Pettik
                   ` (4 more replies)
  0 siblings, 5 replies; 11+ messages in thread
From: Nikita Pettik @ 2019-03-07 13:14 UTC (permalink / raw)
  To: tarantool-patches; +Cc: v.shpilevoy, Nikita Pettik

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

^ permalink raw reply	[flat|nested] 11+ messages in thread

end of thread, other threads:[~2019-03-07 14:14 UTC | newest]

Thread overview: 11+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-07 13:14 [tarantool-patches] [PATCH 0/4] SQL types changes Nikita Pettik
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

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox