[Tarantool-patches] [PATCH vshard 01/11] error: introduce vshard.error.timeout()

Vladislav Shpilevoy v.shpilevoy at tarantool.org
Thu Feb 25 00:46:12 MSK 2021


Hi! Thanks for the review!

On 24.02.2021 11:27, Oleg Babin wrote:
> Hi! Thanks for your patch.
> 
> Personally, I vote for dropping 1.9 support (it's already broken - #256).

It kind of works. Just with some spam in the logs. But yeah, I would like
to finally drop it. I talked to Mons and he approved this:
https://github.com/tarantool/vshard/issues/267

> But if you want to eliminate "long and ugly" ways you could do something like:
> 
> 
> ```
> 
> local make_timeout
> if box.error.new ~= nil then
>     make_timeout = function() return box.error.new(box.error.TIMEOUT) end
> else
>     make_timeout = function() return select(2, pcall(...)) end
> end
> 
> ```

Which is longer and uglier, right? Where is the win?


More information about the Tarantool-patches mailing list