Tarantool development patches archive
 help / color / mirror / Atom feed
From: Mergen Imeev via Tarantool-patches <tarantool-patches@dev.tarantool.org>
To: vdavydov@tarantool.org
Cc: tarantool-patches@dev.tarantool.org
Subject: [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL
Date: Fri, 13 Aug 2021 06:12:52 +0300	[thread overview]
Message-ID: <cover.1628824286.git.imeevma@gmail.com> (raw)

This patch reworks SCALAR and NUMBER types. Afther these patches SCANAR and
NUMBER values will follow newly defined rules. However, only one SQL built-in
function is actually changed in this patch. All other functions that should be
changed will be reworked in issue #6105.

https://github.com/tarantool/tarantool/issues/6221
https://github.com/tarantool/tarantool/tree/imeevma/gh-6221-introduce-mem-type-number

Mergen Imeev (7):
  sql: remove enum field_type from struct Mem
  sql: re-introduce NUMBER and SCALAR meta-types
  sql: disallow implicit cast from NUMBER and SCALAR
  sql: disallow arithmetic for NUMBER and SCALAR
  sql: disallow bitwise for NUMBER and SCALAR
  sql: disallow concatination for SCALAR
  sql: fix comparison with SCALAR value

 .../gh-5956-remove-field-types-from-mem.md    |   3 +
 ...21-re-introduce-scalar-and-number-types.md |   8 +
 src/box/sql/func.c                            |  19 +-
 src/box/sql/mem.c                             | 388 ++++++++----------
 src/box/sql/mem.h                             |  17 +-
 src/box/sql/vdbe.c                            |  35 +-
 src/box/sql/vdbeInt.h                         |   3 -
 src/box/sql/vdbesort.c                        |   6 -
 test/sql-tap/cast.test.lua                    |  26 +-
 test/sql-tap/colname.test.lua                 |   4 +-
 test/sql-tap/e_select1.test.lua               |   6 +-
 test/sql-tap/engine.cfg                       |   3 +
 test/sql-tap/func.test.lua                    |  12 +-
 test/sql-tap/func5.test.lua                   |  75 +++-
 .../gh-5335-wrong-int-to-double-cast.test.lua |  40 --
 ...-5756-implicit-cast-in-arithmetic.test.lua |  32 +-
 test/sql-tap/in4.test.lua                     |   2 +-
 test/sql-tap/metatypes.test.lua               | 187 +++++++++
 test/sql-tap/minmax2.test.lua                 |   2 +-
 test/sql-tap/numcast.test.lua                 |  58 +--
 test/sql-tap/select7.test.lua                 |   8 +-
 test/sql-tap/sort.test.lua                    |  14 +-
 test/sql-tap/sql-errors.test.lua              |   8 +-
 test/sql-tap/table.test.lua                   |   4 +-
 test/sql-tap/tkt-7bbfb7d442.test.lua          |   4 +-
 test/sql-tap/tkt-91e2e8ba6f.test.lua          |  96 -----
 test/sql-tap/tkt-a8a0d2996a.test.lua          |  18 +-
 test/sql-tap/trigger9.test.lua                |   2 +-
 test/sql-tap/uuid.test.lua                    |  12 +-
 test/sql-tap/view.test.lua                    |   4 +-
 test/sql/boolean.result                       | 368 +++++++++--------
 test/sql/boolean.test.sql                     |   4 +-
 test/sql/gh-4697-scalar-bool-sort-cmp.result  |  16 +-
 test/sql/prepared.result                      |   2 +-
 test/sql/prepared.test.lua                    |   2 +-
 test/sql/types.result                         |  19 +-
 36 files changed, 761 insertions(+), 746 deletions(-)
 create mode 100644 changelogs/unreleased/gh-5956-remove-field-types-from-mem.md
 create mode 100644 changelogs/unreleased/gh-6221-re-introduce-scalar-and-number-types.md
 delete mode 100755 test/sql-tap/gh-5335-wrong-int-to-double-cast.test.lua
 create mode 100755 test/sql-tap/metatypes.test.lua
 delete mode 100755 test/sql-tap/tkt-91e2e8ba6f.test.lua

-- 
2.25.1


             reply	other threads:[~2021-08-13  3:12 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-08-13  3:12 Mergen Imeev via Tarantool-patches [this message]
2021-08-13  3:12 ` [Tarantool-patches] [PATCH v1 1/7] sql: remove enum field_type from struct Mem Mergen Imeev via Tarantool-patches
2021-08-13  3:12 ` [Tarantool-patches] [PATCH v1 2/7] sql: re-introduce NUMBER and SCALAR meta-types Mergen Imeev via Tarantool-patches
2021-08-13  3:12 ` [Tarantool-patches] [PATCH v1 3/7] sql: disallow implicit cast from NUMBER and SCALAR Mergen Imeev via Tarantool-patches
2021-08-13  3:13 ` [Tarantool-patches] [PATCH v1 4/7] sql: disallow arithmetic for " Mergen Imeev via Tarantool-patches
2021-08-13  3:13 ` [Tarantool-patches] [PATCH v1 5/7] sql: disallow bitwise " Mergen Imeev via Tarantool-patches
2021-08-13  3:13 ` [Tarantool-patches] [PATCH v1 6/7] sql: disallow concatination for SCALAR Mergen Imeev via Tarantool-patches
2021-08-13  3:13 ` [Tarantool-patches] [PATCH v1 7/7] sql: fix comparison with SCALAR value Mergen Imeev via Tarantool-patches
2021-08-13 16:08 ` [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL Vladimir Davydov via Tarantool-patches
2021-08-17 11:30   ` Vitaliia Ioffe via Tarantool-patches
2021-08-18 12:29 ` Kirill Yukhin via Tarantool-patches
  -- strict thread matches above, loose matches on Subject: below --
2021-08-11 16:01 Mergen Imeev via Tarantool-patches

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.1628824286.git.imeevma@gmail.com \
    --to=tarantool-patches@dev.tarantool.org \
    --cc=imeevma@tarantool.org \
    --cc=vdavydov@tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL' \
    /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