[tarantool-patches] Re: [PATCH 0/2] Box.session.sync

[Vladislav Shpilevoy <v.shpilevoy@tarantool.org>]

Sorry. Forgot the description.

The patchset fixes box.session.sync(). Before the patchset
box.session.sync() is session global, and is changed after
a new request arrives. So box.session.sync() can be safely
used only when a connection behind the session is not
multiplexed.

But box.session.push() can be used in a multiplexed
connection and uses box.session.sync() as a default
sync. So it needs box.session.sync() be request local.

On 09/06/2018 20:49, Vladislav Shpilevoy wrote:
> Branch: http://github.com/tarantool/tarantool/tree/gerold103/gh-3450-session-sync
> Issue: https://github.com/tarantool/tarantool/issues/3450
> 
> Vladislav Shpilevoy (2):
>    fiber: remove fiber local storage
>    session: fix box.session.sync()
> 
>   src/box/iproto.cc             | 11 ++++--
>   src/box/session.cc            |  4 +-
>   src/box/session.h             | 31 ++++++---------
>   src/box/txn.c                 |  2 +-
>   src/box/txn.h                 |  2 +-
>   src/fiber.c                   |  9 +----
>   src/fiber.h                   | 89 ++++++++++++++++++-------------------------
>   src/fiber_channel.c           | 31 +++++----------
>   src/lua/fiber.c               | 19 ++++-----
>   test/box-tap/session.test.lua | 29 +++++++++++++-
>   10 files changed, 106 insertions(+), 121 deletions(-)
>