From: imeevma@tarantool.org
To: korablev@tarantool.org
Cc: tarantool-patches@dev.tarantool.org
Subject: [Tarantool-patches] [PATCH v2 0/3] sql: fix NUMBER type
Date: Mon, 30 Dec 2019 16:01:23 +0300 [thread overview]
Message-ID: <cover.1577710381.git.imeevma@gmail.com> (raw)
This patch-set makes number to be union of UNSIGNED, INTEGER and
DOUBLE numeric types. Also it fixes couple of small bugs of DOUBLE
type.
https://github.com/tarantool/tarantool/issues/4233
https://github.com/tarantool/tarantool/tree/imeevma/gh-4233-fix-number-field-type-in-sql
Mergen Imeev (3):
sql: allow conversion of numeric binary values to DOUBLE
sql: fix typeof() for double values
sql: make NUMBER to be union of SQL numeric types
src/box/sql/func.c | 2 +-
src/box/sql/vdbe.c | 15 +---
src/box/sql/vdbeInt.h | 1 -
src/box/sql/vdbemem.c | 46 ++-----------
test/sql-tap/cast.test.lua | 44 ++++++------
test/sql-tap/check.test.lua | 4 +-
test/sql-tap/e_select1.test.lua | 2 +-
test/sql-tap/func.test.lua | 4 +-
test/sql-tap/numcast.test.lua | 128 ++++++++++++++++++++++++++++++++++-
test/sql-tap/select3.test.lua | 8 +--
test/sql-tap/sort.test.lua | 12 ++--
test/sql-tap/tkt-80e031a00f.test.lua | 8 +--
test/sql-tap/tkt-91e2e8ba6f.test.lua | 12 ++--
test/sql/integer-overflow.result | 2 +-
test/sql/types.result | 9 ++-
test/sql/types.test.lua | 9 +++
16 files changed, 198 insertions(+), 108 deletions(-)
--
2.7.4
next reply other threads:[~2019-12-30 13:01 UTC|newest]
Thread overview: 6+ messages / expand[flat|nested] mbox.gz Atom feed top
2019-12-30 13:01 imeevma [this message]
2019-12-30 13:01 ` [Tarantool-patches] [PATCH v2 1/3] sql: allow conversion of numeric binary values to DOUBLE imeevma
2019-12-30 13:17 ` Nikita Pettik
2019-12-31 8:16 ` Mergen Imeev
2019-12-30 13:01 ` [Tarantool-patches] [PATCH v2 2/3] sql: fix typeof() for double values imeevma
2019-12-30 13:01 ` [Tarantool-patches] [PATCH v2 3/3] sql: make NUMBER to be union of SQL numeric types imeevma
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.1577710381.git.imeevma@gmail.com \
--to=imeevma@tarantool.org \
--cc=korablev@tarantool.org \
--cc=tarantool-patches@dev.tarantool.org \
--subject='Re: [Tarantool-patches] [PATCH v2 0/3] sql: fix NUMBER type' \
/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