[tarantool-patches] [PATCH v2 1/1] box: restore region after box.snapshot()
Vladimir Davydov
vdavydov.dev at gmail.com
Thu Nov 15 14:25:16 MSK 2018
On Wed, Nov 14, 2018 at 01:44:29PM +0300, imeevma at tarantool.org wrote:
> Before this patch region wasn't restored after box.snapshot() so
> some tests fail due to "memory leak". This patch adds additional
> memory management and fixes mentioned failures.
Bad comment. It's unclear what this patch actually does to fix the
issue. Looks like you didn't bother to rewrite it in v2.
I fixed it and pushed the patch to 2.1
>
> Closes #3732
> ---
> Issue: https://github.com/tarantool/tarantool/issues/assigned/ImeevMA
Invalid link. BTW, "Issue:" and "Branch:" are usless.
Also, changelog since v2 is missing. And it would be prudent to put
a link here to v1, like this:
Changes in v2:
- Use malloc() in memtx instead of region_alloc().
- Truncate region in engine callbacks rather than in box_snapshot().
v1: https://www.freelists.org/post/tarantool-patches/PATCH-v1-11-box-restore-region-after-boxsnapshot
> Branch: https://github.com/tarantool/tarantool/tree/imeevma/gh-3732-additional-memory-managament-in-box_snapshot
>
> src/box/memtx_engine.c | 41 ++++++++++++++++++++---------------------
> src/box/vy_log.c | 11 ++++++++---
> test/engine/snapshot.result | 35 +++++++++++++++++++++++++++++++++++
> test/engine/snapshot.test.lua | 17 +++++++++++++++++
> 4 files changed, 80 insertions(+), 24 deletions(-)
More information about the Tarantool-patches
mailing list