[tarantool-patches] [PATCH v1 0/3] sql: make SQL_TARANTOOL_ERROR the only errcode of OP_Halt

imeevma at tarantool.org imeevma at tarantool.org
Fri Apr 12 15:34:15 MSK 2019


Currently there are two ways to set an error in SQL - through
OP_Halt and through abort_due_to_error. After this patch, all
errors set via OP_Halt will be set using diag_set().

https://github.com/tarantool/tarantool/issues/4074
https://github.com/tarantool/tarantool/tree/imeevma/gh-4074-diag_set-in-op_halt

Mergen Imeev (3):
  sql: remove mayAbort field from struct Parse
  sql: rework diag_set() in OP_Halt
  sql: make SQL_TARANTOOL_ERROR the only errcode of OP_Halt

 src/box/sql/build.c             |  57 +-----------------
 src/box/sql/expr.c              |   9 ++-
 src/box/sql/fk_constraint.c     |  42 +------------
 src/box/sql/insert.c            |  31 +++++-----
 src/box/sql/select.c            |  17 +++---
 src/box/sql/sqlInt.h            |   3 -
 src/box/sql/trigger.c           |  12 ++--
 src/box/sql/vdbe.c              |  24 ++------
 src/box/sql/vdbe.h              |   3 -
 src/box/sql/vdbeaux.c           | 127 ----------------------------------------
 test/sql-tap/e_select1.test.lua |   4 +-
 test/sql-tap/limit.test.lua     |  26 ++++----
 test/sql-tap/select4.test.lua   |   4 +-
 test/sql/iproto.result          |  18 ++++--
 14 files changed, 68 insertions(+), 309 deletions(-)

-- 
2.7.4





More information about the Tarantool-patches mailing list