Tarantool development patches archive
 help / color / mirror / Atom feed
From: Timur Safin <tsafin@tarantool.org>
To: korablev@tarantool.org, imeevma@tarantool.org
Cc: tarantool-patches@dev.tarantool.org
Subject: [Tarantool-patches] [PATCH v1 0/2] Make TPC-H run again
Date: Thu, 30 Apr 2020 11:26:52 +0300	[thread overview]
Message-ID: <cover.1588234865.git.tsafin@tarantool.org> (raw)

NB! This is not going to be submitted, and I'm sending this 
patch-set to just make visible all necessary patches, to keep 
it for later usage with TPC-H queries we ought to run regularly.

We restore preliminary DATE types support (right now that would 
mean simple STRING) with a set of built-in SQL functions, which
were necessary to make run big chunk (13 out of 22) of SQL TPC-H 
benchmark queries. 

There is opinion that we do not want make date support available
as officially supported, to avoid future data migrations, once we 
will complete proper datetime types support based on decimals. 
So I'll keep these patches handy for foresable future...

Changes in v1:
Nothing substantial - just trying to link commits with original
GitHub issues #4898

GH issue: https://github.com/tarantool/tarantool/issues/4898
GitHub branch:
https://github.com/tarantool/tarantool/tree/tsafin/gh-4898-sql-datetime-mocks


Timur Safin (2):
  Restore partial date/time support in Tarantool SQL
  Restore date/datetime tests

 extra/mkkeywordhash.c                |  10 +--
 src/box/sql/date.c                   |  70 +++------------
 src/box/sql/dateInt.h                | 122 +++++++++++++++++++++++++++
 src/box/sql/func.c                   |  49 +++++------
 src/box/sql/parse.y                  |  27 +++---
 test/sql-tap/date.test.lua           |   4 -
 test/sql-tap/keyword1.test.lua       |   5 +-
 test/sql-tap/table.test.lua          |   6 +-
 test/sql-tap/tkt-7bbfb7d442.test.lua |   4 +-
 test/sql-tap/tkt-bd484a090c.test.lua |   6 +-
 test/sql-tap/tkt2192.test.lua        |   6 +-
 test/sql-tap/tkt3791.test.lua        |   5 +-
 test/sql/triggers.result             |   4 +-
 test/sql/triggers.test.lua           |   4 +-
 14 files changed, 186 insertions(+), 136 deletions(-)
 create mode 100644 src/box/sql/dateInt.h

-- 
2.20.1

             reply	other threads:[~2020-04-30  8:27 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-04-30  8:26 Timur Safin [this message]
2020-04-30  8:26 ` [Tarantool-patches] [PATCH v1 1/2] Restore partial date/time support in Tarantool SQL Timur Safin
2020-04-30  8:26 ` [Tarantool-patches] [PATCH v1 2/2] Restore date/datetime tests Timur Safin

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.1588234865.git.tsafin@tarantool.org \
    --to=tsafin@tarantool.org \
    --cc=imeevma@tarantool.org \
    --cc=korablev@tarantool.org \
    --cc=tarantool-patches@dev.tarantool.org \
    --subject='Re: [Tarantool-patches] [PATCH v1 0/2] Make TPC-H run again' \
    /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