Tarantool development patches archive
 help / color / mirror / Atom feed
* [tarantool-patches] [PATCH v4 0/8] sql: use diag_set() for errors in SQL
@ 2019-03-13 17:03 imeevma
  2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 1/8] sql: rework syntax errors imeevma
                   ` (7 more replies)
  0 siblings, 8 replies; 43+ messages in thread
From: imeevma @ 2019-03-13 17:03 UTC (permalink / raw)
  To: korablev; +Cc: tarantool-patches

The goal of this patch-set is to rework all SQL errors and make
Tarantool ones from them. Currently this patch-set contains pathes
that reworks all parser errors.

https://github.com/tarantool/tarantool/issues/3965
https://github.com/tarantool/tarantool/tree/imeevma/gh-3965-use-diag_set-to-describe-error

Difference from last version:
 - Some patches were removed and new added.
 - A bit changed position of patches in the set.

Mergen Imeev (8):
  sql: rework syntax errors
  sql: set SQL parser errors via diag_set()
  sql: replace rc with is_aborted status in struct Parse
  sql: remove field nErr from struct Parse
  sql: remove field zErrMsg from struct Parse
  sql: rework three errors of "unsupported" type
  sql: rework semantic errors
  sql: remove sqlErrorMsg()

 src/box/errcode.h                                  |  18 +-
 src/box/sql.c                                      |  12 +-
 src/box/sql/alter.c                                |   3 +-
 src/box/sql/analyze.c                              |  13 +-
 src/box/sql/build.c                                | 221 +++++-----
 src/box/sql/callback.c                             |   3 +-
 src/box/sql/delete.c                               |  31 +-
 src/box/sql/expr.c                                 | 175 ++++----
 src/box/sql/insert.c                               |  40 +-
 src/box/sql/malloc.c                               |   9 +-
 src/box/sql/parse.y                                |  74 ++--
 src/box/sql/pragma.c                               |  11 +-
 src/box/sql/prepare.c                              |  22 +-
 src/box/sql/resolve.c                              | 279 ++++++++-----
 src/box/sql/select.c                               | 219 +++++-----
 src/box/sql/sqlInt.h                               |  30 +-
 src/box/sql/tokenize.c                             |  84 ++--
 src/box/sql/trigger.c                              |  35 +-
 src/box/sql/update.c                               |  22 +-
 src/box/sql/util.c                                 |  39 --
 src/box/sql/vdbemem.c                              |   8 +-
 src/box/sql/where.c                                |  13 +-
 src/box/sql/wherecode.c                            |   2 +-
 src/box/sql/whereexpr.c                            |   8 +-
 test/box/alter.result                              |   4 +-
 test/box/misc.result                               |  15 +
 test/sql-tap/alter2.test.lua                       |   4 +-
 test/sql-tap/analyzeD.test.lua                     |   2 +-
 test/sql-tap/autoinc.test.lua                      |   2 +-
 test/sql-tap/blob.test.lua                         |  20 +-
 test/sql-tap/check.test.lua                        |  16 +-
 test/sql-tap/colname.test.lua                      |   4 +-
 test/sql-tap/count.test.lua                        |   2 +-
 test/sql-tap/default.test.lua                      |  14 +-
 test/sql-tap/e_select1.test.lua                    |  36 +-
 test/sql-tap/engine.cfg                            |   3 +
 test/sql-tap/fkey2.test.lua                        |   4 +-
 test/sql-tap/func3.test.lua                        |   6 +-
 test/sql-tap/gh-2549-many-columns.test.lua         |   2 +-
 test/sql-tap/gh2168-temp-tables.test.lua           |   4 +-
 test/sql-tap/gh2548-select-compound-limit.test.lua |   2 +-
 test/sql-tap/hexlit.test.lua                       |   4 +-
 test/sql-tap/identifier_case.test.lua              |   2 +-
 test/sql-tap/index-info.test.lua                   |   2 +-
 test/sql-tap/index1.test.lua                       |   2 +-
 test/sql-tap/index7.test.lua                       |   2 +-
 test/sql-tap/insert1.test.lua                      |   2 +-
 test/sql-tap/join.test.lua                         |  22 +-
 test/sql-tap/join3.test.lua                        |   2 +-
 test/sql-tap/keyword1.test.lua                     |   2 +-
 test/sql-tap/misc1.test.lua                        |  14 +-
 test/sql-tap/misc5.test.lua                        |   6 +-
 test/sql-tap/null.test.lua                         |  12 +-
 test/sql-tap/pragma.test.lua                       |   4 +-
 test/sql-tap/select1.test.lua                      |  26 +-
 test/sql-tap/select3.test.lua                      |   8 +-
 test/sql-tap/select4.test.lua                      |   6 +-
 test/sql-tap/select7.test.lua                      |   2 +-
 test/sql-tap/sql-errors.test.lua                   | 459 +++++++++++++++++++++
 test/sql-tap/start-transaction.test.lua            |  14 +-
 test/sql-tap/table.test.lua                        |  10 +-
 test/sql-tap/tkt2822.test.lua                      |  30 +-
 test/sql-tap/tkt3935.test.lua                      |  14 +-
 test/sql-tap/tokenize.test.lua                     |  26 +-
 test/sql-tap/trigger1.test.lua                     |  14 +-
 test/sql-tap/unique.test.lua                       |   2 +-
 test/sql-tap/view.test.lua                         |  24 +-
 test/sql-tap/where7.test.lua                       |   2 +-
 test/sql-tap/whereG.test.lua                       |   6 +-
 test/sql-tap/with1.test.lua                        |  12 +-
 test/sql-tap/with2.test.lua                        |  18 +-
 test/sql/checks.result                             |  12 +-
 test/sql/collation.result                          |  12 +-
 test/sql/foreign-keys.result                       |  10 +-
 test/sql/gh-2347-max-int-literals.result           |   6 +-
 test/sql/gh-2929-primary-key.result                |   8 +-
 test/sql/gh-3888-values-blob-assert.result         |   8 +-
 test/sql/integer-overflow.result                   |   9 +-
 test/sql/iproto.result                             |  16 +-
 test/sql/misc.result                               |  14 +-
 test/sql/on-conflict.result                        |  16 +-
 test/sql/transition.result                         |   3 +-
 test/sql/triggers.result                           |   4 +-
 test/sql/types.result                              |  13 +-
 test/sql/view.result                               |   2 +-
 85 files changed, 1434 insertions(+), 959 deletions(-)
 create mode 100755 test/sql-tap/sql-errors.test.lua

-- 
2.7.4

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

end of thread, other threads:[~2019-03-27  6:49 UTC | newest]

Thread overview: 43+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2019-03-13 17:03 [tarantool-patches] [PATCH v4 0/8] sql: use diag_set() for errors in SQL imeevma
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 1/8] sql: rework syntax errors imeevma
2019-03-14 18:24   ` [tarantool-patches] " n.pettik
2019-03-14 18:28     ` Imeev Mergen
2019-03-15 14:09   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 2/8] sql: set SQL parser errors via diag_set() imeevma
2019-03-14 19:26   ` [tarantool-patches] " n.pettik
2019-03-14 19:36     ` n.pettik
2019-03-18 15:06     ` Mergen Imeev
2019-03-19  9:41       ` n.pettik
2019-03-19 11:24   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 3/8] sql: replace rc with is_aborted status in struct Parse imeevma
2019-03-14 19:53   ` [tarantool-patches] " n.pettik
2019-03-18 15:28     ` Mergen Imeev
2019-03-19  9:54       ` n.pettik
2019-03-19 13:17   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 4/8] sql: remove field nErr from " imeevma
2019-03-14 19:58   ` [tarantool-patches] " n.pettik
2019-03-19 13:27   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 5/8] sql: remove field zErrMsg " imeevma
2019-03-14 22:15   ` [tarantool-patches] " n.pettik
2019-03-19 13:20   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 6/8] sql: rework three errors of "unsupported" type imeevma
2019-03-14 22:15   ` [tarantool-patches] " n.pettik
2019-03-19 13:30   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 7/8] sql: rework semantic errors imeevma
2019-03-15 15:49   ` [tarantool-patches] " n.pettik
2019-03-22 12:48     ` Mergen Imeev
2019-03-26 14:14       ` n.pettik
2019-03-26 16:56         ` Mergen Imeev
2019-03-26 18:16           ` n.pettik
2019-03-26 19:20             ` Mergen Imeev
2019-03-26 21:36               ` n.pettik
2019-03-27  6:48   ` Kirill Yukhin
2019-03-13 17:03 ` [tarantool-patches] [PATCH v4 8/8] sql: remove sqlErrorMsg() imeevma
2019-03-15 13:36   ` [tarantool-patches] " n.pettik
2019-03-25 18:47     ` Mergen Imeev
2019-03-26 13:34       ` n.pettik
2019-03-26 17:52         ` Mergen Imeev
2019-03-26 18:28           ` n.pettik
2019-03-26 19:21             ` Mergen Imeev
2019-03-26 21:36               ` n.pettik
2019-03-27  6:49   ` Kirill Yukhin

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