[Tarantool-patches] [PATCH v2 3/5] error: Increase the number of fields transmitted through IPROTO
Vladislav Shpilevoy
v.shpilevoy at tarantool.org
Thu Apr 16 03:02:35 MSK 2020
Thanks for the fixes!
>>> diff --git a/src/box/lua/net_box.lua b/src/box/lua/net_box.lua
>>> index 07fa54c..1e0cd7a 100644
>>> --- a/src/box/lua/net_box.lua
>>> +++ b/src/box/lua/net_box.lua
>>> @@ -287,7 +289,24 @@ local function create_transport(host, port, user, password, callback,
>>> local error = self.response[i]
>>> local code = error[IPROTO_ERROR_CODE]
>>> local msg = error[IPROTO_ERROR_MESSAGE]
>>> - local new_err = box.error.new({code = code, reason = msg})
>>> + local custom_type = error[IPROTO_ERROR_CUSTOM_TYPE]
>>> + local traceback = error[IPROTO_ERROR_TRACEBACK]
> After we decided don't concatenate the traceback, I decided that it would be inconsistent to transmit the traceback in the case of "throw" over the network, since when creating a new error, the "trace" is set new and it always has been. Furthermore, an error can change the type. So, I delete IPROTO_ERROR_TRACEBACK.If you consider it wrong decision, I can return IPROTO_ERROR_TRACEBACK.
I am ok with that. As I said, traceback is not really a kind of
critical and stable feature anyway.
More information about the Tarantool-patches
mailing list