[tarantool-patches] [PATCH v3 0/6] box: stacked diagnostics area in fiber
Kirill Shcherbatov
kshcherbatov at tarantool.org
Mon Sep 2 17:51:08 MSK 2019
Support stacked diagnostics for Tarantool allows to accumulate all
occurred errors during processing a request. This allows to better
understand what has happened and handle errors correspondingly.
This patch introduces core methods diag_add(), diag_svp(),
diag_rollback_to_svp() that allows you to produce a better
diagnostics information for a user.
There is an other new Lua endpoint
box.error.prev() that allows you to get a reason for a given
error object.
Changes in v3:
- fixed Kostya's comments
- implemented iproto support for stacked diagnostics
Changes in v2:
- new API - user shouldn't have an ability to modify an existent
error object that he didn't constructed.
- a new detailed unit-test that performs a good coverage for
many complicated reference-counting scenarios.
Branch: http://github.com/tarantool/tarantool/tree/kshch/gh-1148-stacked-errors
Issue: https://github.com/tarantool/tarantool/issues/1148
Kirill Shcherbatov (6):
box: rfc for stacked diagnostic area in Tarantool
box: rename diag_add_error to diag_set_error
box: introduce stacked diagnostic area
box: extend box.error.new({}) API
iproto: refactor error encoding with mpstream
iproto: support stacked diagnostics for errors
src/box/error.h | 5 +-
src/box/iproto_constants.h | 1 +
src/lib/core/diag.h | 84 +++++++++++-
src/lib/core/exception.h | 2 +-
src/lua/error.h | 3 +
src/box/key_list.c | 16 +--
src/box/lua/call.c | 6 +-
src/box/sql/vdbe.c | 2 +-
src/box/vy_scheduler.c | 6 +-
src/box/xrow.c | 198 +++++++++++++++++++++++-----
src/lib/core/diag.c | 1 +
src/lua/error.c | 2 +-
src/lua/utils.c | 2 +-
test/app-tap/module_api.c | 3 +-
test/box/function1.c | 29 ++--
test/box/reload1.c | 20 +--
test/box/reload2.c | 9 +-
test/box/tuple_bench.c | 10 +-
doc/rfc/1148-stacked-diagnostics.md | 180 +++++++++++++++++++++++++
src/box/applier.cc | 2 +-
src/box/error.cc | 7 +-
src/box/lua/error.cc | 35 ++++-
src/box/lua/net_box.lua | 24 +++-
src/box/relay.cc | 4 +-
src/lib/core/exception.cc | 2 +-
src/lua/error.lua | 12 ++
test/app/fiber.result | 7 +-
test/box-py/iproto.result | 6 +-
test/box-py/iproto.test.py | 6 +-
test/box/misc.result | 159 +++++++++++++++++++++-
test/box/misc.test.lua | 49 +++++++
test/engine/func_index.result | 52 ++++++--
test/engine/func_index.test.lua | 7 +
test/unit/CMakeLists.txt | 2 +-
test/unit/fiber.cc | 157 ++++++++++++++++++++++
test/unit/fiber.result | 60 +++++++++
36 files changed, 1045 insertions(+), 125 deletions(-)
create mode 100644 doc/rfc/1148-stacked-diagnostics.md
--
2.22.1
More information about the Tarantool-patches
mailing list