[tarantool-patches] [PATCH v3 0/5] refactoring: remove exceptions from triggers

Ilya Kosarev i.kosarev at tarantool.org
Wed Sep 11 16:05:14 MSK 2019


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

Changes in v3:
1) Style fixes: switch .. case, indentation
2) In some patches extra diag_raise calls are removed

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 (5):
  refactoring: remove exceptions from triggers except alter.cc
  refactoring: remove exceptions from used in alter.cc outer functions
  refactoring: replace exceptions in most alter.cc  functions
  refactoring: remove obvious exceptions in alter.cc
  refactoring: replace remaining exceptions in alter.cc & update
    comments

 src/box/alter.cc            | 2704 ++++++++++++++++++++++-------------
 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      |   75 +-
 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/box/vy_scheduler.c      |    3 +-
 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 +-
 31 files changed, 1887 insertions(+), 1227 deletions(-)

-- 
2.17.1





More information about the Tarantool-patches mailing list