[PATCH v5 2/4] box: introduce field_map_builder class
Vladimir Davydov
vdavydov.dev at gmail.com
Mon May 6 17:22:13 MSK 2019
On Mon, May 06, 2019 at 02:57:39PM +0300, Kirill Shcherbatov wrote:
> The new field_map_builder class encapsulates the logic associated
> with field_map allocation and initialization. In the future it
> will be extended to allocate field_map that has extensions.
>
> Needed for #1257
> ---
> src/box/CMakeLists.txt | 1 +
> src/box/field_map.c | 61 ++++++++++++++++++
> src/box/field_map.h | 140 +++++++++++++++++++++++++++++++++++++++++
> src/box/memtx_engine.c | 8 +--
> src/box/sql.c | 5 +-
> src/box/tuple.c | 14 ++---
> src/box/tuple.h | 6 +-
> src/box/tuple_format.c | 30 +++------
> src/box/tuple_format.h | 12 ++--
> src/box/vy_stmt.c | 9 +--
> 10 files changed, 238 insertions(+), 48 deletions(-)
> create mode 100644 src/box/field_map.c
> create mode 100644 src/box/field_map.h
Looks good to me. I'll commit as soon as the questions with patch #1
have been resolved.
More information about the Tarantool-patches
mailing list