[PATCH v2 2/4] Set is_temporary flag for formats of ephemeral spaces

Vladimir Davydov vdavydov.dev at gmail.com
Thu Jan 24 20:26:31 MSK 2019


On Thu, Jan 24, 2019 at 03:48:46PM +0300, Kirill Yukhin wrote:
> Before the patch, when ephemeral space was created flag
> is_temporary was set after space was actually created.
> Which in turn lead to corresponding flag of tuple_format
> being set to `false`.
> So, having heavy load using ephemeral spaces (almost any
> SQL query) and snapshotting at the same time might lead
> to OOM, since tuples of ephemeral spaces were not marked
> as temporary and were not gc-ed.
> Patch sets the flag in space definition.
> ---
>  src/box/space.c     |  2 +-
>  src/box/space_def.c | 14 ++++++++++++++
>  src/box/space_def.h |  9 +++++++++
>  src/box/sql.c       |  6 +-----
>  4 files changed, 25 insertions(+), 6 deletions(-)

Pushed to 2.1



More information about the Tarantool-patches mailing list