From: Vladislav Shpilevoy via Tarantool-patches <tarantool-patches@dev.tarantool.org> To: tarantool-patches@dev.tarantool.org, sergepetrenko@tarantool.org Subject: [Tarantool-patches] [PATCH v2 00/11] ER_READONLY reason Date: Fri, 12 Nov 2021 00:54:19 +0100 [thread overview] Message-ID: <cover.1636674803.git.v.shpilevoy@tarantool.org> (raw) Changes in v2: - A lot of minor fixes in all the commits; - libuuid is moved into libcore; - ER_READONLY message got the same details as in the fields. Branch: http://github.com/tarantool/tarantool/tree/gerold103/gh-5568-err-readonly-reason Issue: https://github.com/tarantool/tarantool/issues/5568 Vladislav Shpilevoy (11): diag: return created error from diag_set() uuid: move into libcore error: introduce error_payload error: move code to struct error from ClientError error: use error_payload to store optional members error: use error_payload in MessagePack codecs error: use error_payload in Lua luatest: copy config in cluster:build_server() luatest: add new helpers for 'server' object box: enrich ER_READONLY with new details error: report ER_READONLY reason in message .../unreleased/gh-5568-readonly-reason.md | 4 + extra/exports | 3 +- src/CMakeLists.txt | 3 +- src/box/applier.h | 2 +- src/box/bind.h | 2 +- src/box/box.cc | 69 ++- src/box/errcode.h | 2 +- src/box/error.cc | 57 +-- src/box/error.h | 60 +-- src/box/field_def.c | 4 +- src/box/index.cc | 4 +- src/box/lua/serialize_lua.c | 2 +- src/box/mp_error.cc | 197 +++----- src/box/msgpack.c | 2 +- src/box/replication.h | 2 +- src/box/sql/mem.c | 2 +- src/box/sql/mem.h | 2 +- src/box/tuple.h | 2 +- src/box/tuple_compare.cc | 8 +- src/box/xlog.h | 2 +- src/box/xrow.h | 2 +- src/lib/CMakeLists.txt | 1 - src/lib/core/CMakeLists.txt | 3 + src/lib/core/crash.c | 2 +- src/lib/core/diag.c | 37 +- src/lib/core/diag.h | 109 +++- src/lib/core/error_payload.c | 301 +++++++++++ src/lib/core/error_payload.h | 190 +++++++ src/lib/core/exception.cc | 8 +- src/lib/core/exception.h | 66 +++ src/lib/{uuid => core}/mp_uuid.c | 0 src/lib/{uuid => core}/mp_uuid.h | 0 src/lib/{uuid => core}/tt_uuid.c | 0 src/lib/{uuid => core}/tt_uuid.h | 0 src/lib/mpstream/CMakeLists.txt | 2 +- src/lib/mpstream/mpstream.c | 2 +- src/lib/swim/CMakeLists.txt | 2 +- src/lib/swim/swim_io.h | 2 +- src/lib/swim/swim_proto.h | 2 +- src/lib/uuid/CMakeLists.txt | 2 - src/lua/error.lua | 84 ++-- src/lua/init.lua | 24 - src/lua/msgpack.c | 6 +- src/lua/tnt_msgpuck.h | 2 +- src/lua/utils.c | 2 +- test/box/error.result | 4 +- test/box/error.test.lua | 2 +- test/engine/func_index.result | 3 +- test/luatest_helpers/cluster.lua | 1 + test/luatest_helpers/server.lua | 66 ++- .../gh_5568_read_only_reason_test.lua | 304 ++++++++++++ test/replication/anon.result | 6 +- test/replication/catch.result | 4 +- test/replication/election_qsync.result | 11 +- test/replication/election_qsync.test.lua | 4 +- .../gh-6034-qsync-limbo-ownership.result | 22 +- .../gh-6034-qsync-limbo-ownership.test.lua | 8 +- test/sql-tap/CMakeLists.txt | 2 +- test/sql-tap/gh-6024-funcs-return-bin.c | 2 +- test/sql-tap/sql_uuid.c | 2 +- test/unit/CMakeLists.txt | 4 +- test/unit/error.c | 466 ++++++++++++++++++ test/unit/error.result | 161 ++++++ test/unit/mp_error.cc | 76 ++- test/unit/mp_error.result | 27 +- test/unit/swim_proto.c | 2 +- test/unit/swim_test_utils.c | 2 +- test/unit/swim_test_utils.h | 2 +- test/unit/uuid.c | 4 +- test/unit/vy_iterators_helper.c | 2 +- test/unit/xrow.cc | 2 +- test/vinyl/misc.result | 2 +- third_party/lua-cjson/lua_cjson.c | 2 +- third_party/lua-yaml/lyaml.cc | 2 +- 74 files changed, 2069 insertions(+), 404 deletions(-) create mode 100644 changelogs/unreleased/gh-5568-readonly-reason.md create mode 100644 src/lib/core/error_payload.c create mode 100644 src/lib/core/error_payload.h rename src/lib/{uuid => core}/mp_uuid.c (100%) rename src/lib/{uuid => core}/mp_uuid.h (100%) rename src/lib/{uuid => core}/tt_uuid.c (100%) rename src/lib/{uuid => core}/tt_uuid.h (100%) delete mode 100644 src/lib/uuid/CMakeLists.txt create mode 100644 test/replication-luatest/gh_5568_read_only_reason_test.lua create mode 100644 test/unit/error.c create mode 100644 test/unit/error.result -- 2.24.3 (Apple Git-128)
next reply other threads:[~2021-11-11 23:54 UTC|newest] Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top 2021-11-11 23:54 Vladislav Shpilevoy via Tarantool-patches [this message] 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 01/11] diag: return created error from diag_set() Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 10/11] box: enrich ER_READONLY with new details Vladislav Shpilevoy via Tarantool-patches 2021-11-12 7:30 ` Serge Petrenko via Tarantool-patches 2021-11-12 23:24 ` Vladislav Shpilevoy via Tarantool-patches 2021-11-15 6:51 ` Serge Petrenko via Tarantool-patches 2021-11-15 21:56 ` Vladislav Shpilevoy via Tarantool-patches 2021-11-16 9:53 ` Serge Petrenko via Tarantool-patches 2021-11-16 22:08 ` Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 11/11] error: report ER_READONLY reason in message Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 02/11] uuid: move into libcore Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 03/11] error: introduce error_payload Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 04/11] error: move code to struct error from ClientError Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 05/11] error: use error_payload to store optional members Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 06/11] error: use error_payload in MessagePack codecs Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 07/11] error: use error_payload in Lua Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 08/11] luatest: copy config in cluster:build_server() Vladislav Shpilevoy via Tarantool-patches 2021-11-11 23:54 ` [Tarantool-patches] [PATCH v2 09/11] luatest: add new helpers for 'server' object Vladislav Shpilevoy via Tarantool-patches 2021-11-12 23:25 ` [Tarantool-patches] [PATCH v2 12/11] error: introduce box.info.ro_reason Vladislav Shpilevoy via Tarantool-patches 2021-11-15 6:53 ` Serge Petrenko via Tarantool-patches 2021-11-16 22:08 ` [Tarantool-patches] [PATCH v2 00/11] ER_READONLY reason Vladislav Shpilevoy via Tarantool-patches
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=cover.1636674803.git.v.shpilevoy@tarantool.org \ --to=tarantool-patches@dev.tarantool.org \ --cc=sergepetrenko@tarantool.org \ --cc=v.shpilevoy@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH v2 00/11] ER_READONLY reason' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox