Tarantool development patches archive
 help / color / mirror / Atom feed
* [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL
@ 2021-08-11 16:01 Mergen Imeev via Tarantool-patches
  2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 1/7] sql: remove enum field_type from struct Mem Mergen Imeev via Tarantool-patches
                   ` (6 more replies)
  0 siblings, 7 replies; 13+ messages in thread
From: Mergen Imeev via Tarantool-patches @ 2021-08-11 16:01 UTC (permalink / raw)
  To: v.shpilevoy; +Cc: tarantool-patches

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                            |  18 +-
 src/box/sql/mem.c                             | 388 ++++++++----------
 src/box/sql/mem.h                             |  17 +-
 src/box/sql/vdbe.c                            |  33 +-
 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, 758 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


^ permalink raw reply	[flat|nested] 13+ messages in thread
* [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL
@ 2021-08-13  3:12 Mergen Imeev via Tarantool-patches
  2021-08-13  3:12 ` [Tarantool-patches] [PATCH v1 1/7] sql: remove enum field_type from struct Mem Mergen Imeev via Tarantool-patches
  0 siblings, 1 reply; 13+ messages in thread
From: Mergen Imeev via Tarantool-patches @ 2021-08-13  3:12 UTC (permalink / raw)
  To: vdavydov; +Cc: tarantool-patches

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


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

end of thread, other threads:[~2021-08-13  3:13 UTC | newest]

Thread overview: 13+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-08-11 16:01 [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 1/7] sql: remove enum field_type from struct Mem Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 2/7] sql: re-introduce NUMBER and SCALAR meta-types Mergen Imeev via Tarantool-patches
2021-08-12 18:51   ` Vladislav Shpilevoy via Tarantool-patches
2021-08-12 22:22     ` Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 3/7] sql: disallow implicit cast from NUMBER and SCALAR Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 4/7] sql: disallow arithmetic for " Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 5/7] sql: disallow bitwise " Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 6/7] sql: disallow concatination for SCALAR Mergen Imeev via Tarantool-patches
2021-08-11 16:01 ` [Tarantool-patches] [PATCH v1 7/7] sql: fix comparison with SCALAR value Mergen Imeev via Tarantool-patches
2021-08-12 18:50   ` Vladislav Shpilevoy via Tarantool-patches
2021-08-12 22:23     ` Mergen Imeev via Tarantool-patches
2021-08-13  3:12 [Tarantool-patches] [PATCH v1 0/7] Rework SCALAR and NUMBER types in SQL Mergen Imeev via Tarantool-patches
2021-08-13  3:12 ` [Tarantool-patches] [PATCH v1 1/7] sql: remove enum field_type from struct Mem Mergen Imeev via Tarantool-patches

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