[Tarantool-patches] [PATCH v2] iproto: add an empty body to the unprepare response

Alexander Turenko alexander.turenko at tarantool.org
Fri Mar 6 23:39:28 MSK 2020


> > Technically there are two ways to extract helpers from a 'core =
> > tarantool' test:
> > 
> > * Add it to, say, test/box/box.lua and to _G.protected_globals.
> > * Add it to a separate Lua file in test/box/lua and to 'lua_libs' field
> >  in test/box/suite.ini. After this you can use `require` for this
> >  module in a test.
> 
> This also seems like a fine solution, if we are to stick to the SOP, I will do this.

It is our process. We should follow it or at least provide strong
reasoning why we shouldn't in a particular case.

> However, I’m not sure whether this patch fixes a bug, it is stated in the code
> that there’s nothing to send in case of unprepare, perhaps it is a feature?

The issue is marked with 'bug' label.

'unprepare' is the only request, which does not contain a body. In
1.6.4-128-gf433f4b0a (see [1]) it was decided to always send a body in a
response and 'unprepare' does not fit this property. It is unexpected
behaviour and it is what we usually call a bug.

[1]: https://github.com/tarantool/tarantool/commit/f433f4b0aeab48be9b5259b6c5065091dbf07553

WBR, Alexander Turenko.


More information about the Tarantool-patches mailing list