[Tarantool-patches] [PATCH v2 0/2] box: fix session.push formatting

Chris Sosnin k.sosnin at tarantool.org
Wed Apr 8 00:52:31 MSK 2020


I’ve sent the patch set to Nikita with the following change log:

@ChangeLog
- Add Lua output format support for box.session.push

I thought this will suffice.

> On 7 Apr 2020, at 23:29, Vladislav Shpilevoy <v.shpilevoy at tarantool.org> wrote:
> 
> Hi! I suggest to hurry up, we have the release closer and closer.
> 
> On 04/04/2020 01:42, Vladislav Shpilevoy wrote:
>> Thanks for the fixes!
>> 
>> The patchset LGTM. Except that the changelog is still
>> missing.
>> 
>> Please, add a changelog, and send on a second review.
>> I propose either Nikita or Alexander L.
>> 
>> On 13/03/2020 15:58, Chris Sosnin wrote:
>>> issue: https://github.com/tarantool/tarantool/issues/4686
>>> branch: https://github.com/tarantool/tarantool/tree/ksosnin/gh-4686-session-push-fmt
>>> 
>>> Changes in v2:
>>> - Current format moved to struct session.
>>> - Applied review fixes.
>>> 
>>> Chris Sosnin (2):
>>>  refactoring: store output format in struct session
>>>  box: fix formatting in session.push
>>> 
>>> extra/exports                 |  2 +
>>> src/box/lua/console.c         | 53 ++++++++++++++++++--------
>>> src/box/lua/console.lua       | 72 +++++++++++++++++++++++++++--------
>>> src/box/session.h             | 22 ++++++++---
>>> test/app-tap/console.test.lua | 14 ++++++-
>>> 5 files changed, 126 insertions(+), 37 deletions(-)
>>> 



More information about the Tarantool-patches mailing list