[tarantool-patches] [PATCH v2 0/6] refactoring: remove exceptions from triggers

Ilya Kosarev i.kosarev at tarantool.org
Fri Aug 16 21:37:46 MSK 2019


Triggers don't throw exceptions any more. Now they have
return codes to report errors.

Changes in v2:
1) Patch is divided into parts.
2) Corresponding comments are updated according to code changes.

Branch: https://github.com/tarantool/tarantool/tree/i.kosarev/gh-4247-remove-exceptions-from-triggers 
Issue: https://github.com/tarantool/tarantool/issues/4247

Ilya Kosarev (6):
  refactoring: remove exceptions from triggers except alter.cc
  refactoring: remove exceptions from used in alter.cc outer functions
  refactoring: replace most obvious exceptions in alter.cc
  refactoring: replace exceptions in most alter.cc functions
  refactoring: replace some more exceptions in alter.cc
  refactoring: replace remaining exceptions in alter.cc & update
    comments

 src/box/alter.cc            | 3297 +++++++++++++++++++++--------------
 src/box/applier.cc          |   18 +-
 src/box/ck_constraint.c     |    9 +-
 src/box/ck_constraint.h     |    4 +-
 src/box/identifier.h        |   10 -
 src/box/iproto.cc           |    3 +-
 src/box/lua/call.c          |    5 +-
 src/box/lua/sequence.c      |    3 +-
 src/box/lua/space.cc        |    3 +-
 src/box/memtx_space.c       |   17 +-
 src/box/relay.cc            |    5 +-
 src/box/replication.cc      |  193 +-
 src/box/replication.h       |    2 +-
 src/box/schema.cc           |   28 +-
 src/box/schema.h            |   17 +-
 src/box/sequence.h          |    9 -
 src/box/session.cc          |    3 +-
 src/box/tuple.h             |   69 -
 src/box/txn.c               |   10 +-
 src/box/user.cc             |    9 +-
 src/box/user.h              |   14 +-
 src/box/vinyl.c             |   44 +-
 src/lib/core/fiber.c        |    3 +-
 src/lib/core/trigger.cc     |   23 +-
 src/lib/core/trigger.h      |    2 +-
 src/lua/trigger.c           |    7 +-
 src/main.cc                 |    3 +-
 test/unit/cbus.c            |    3 +-
 test/unit/swim.c            |    6 +-
 test/unit/swim_test_utils.c |    3 +-
 30 files changed, 2240 insertions(+), 1582 deletions(-)

-- 
2.17.1





More information about the Tarantool-patches mailing list