[tarantool-patches] Re: [PATCH v3 0/5] refactoring: remove exceptions from triggers
Georgy Kirichenko
georgy at tarantool.org
Wed Sep 18 21:56:44 MSK 2019
I'm sorry but the patchset is mostly unreviewable and unmergeable.
Please split each patch into small independent parts, for instance -
1. txn_alter_trigger_new doesn't throw
2. tnt_raise -> diag_set() return -1; at some 1-2-3 triggers
3. use non _xc version of function in some triggers
4 ....
So we could review and merge a big count of small diffs even if you made
mistakes in others.
Additionally you forgot about some exceptions like fiber_testcancel <-
replica_on_applier_connect <- replica_on_applier_state_f
Also, please don't change code indentation without any reason.
I fixed the first patch and attached diff to the reply, you could rebase your
changes against it.
On Wednesday, September 11, 2019 4:05:14 PM MSK Ilya Kosarev wrote:
> 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-except
> ions-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(-)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.diff
Type: text/x-patch
Size: 48693 bytes
Desc: not available
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20190918/df7c20e9/attachment.bin>
More information about the Tarantool-patches
mailing list