From: Kirill Shcherbatov <kshcherbatov@tarantool.org> To: tarantool-patches@freelists.org, vdavydov.dev@gmail.com Cc: kostja@tarantool.org, Kirill Shcherbatov <kshcherbatov@tarantool.org> Subject: [PATCH v6 0/8] box: Indexes by JSON path Date: Mon, 17 Dec 2018 09:52:44 +0300 [thread overview] Message-ID: <cover.1544995259.git.kshcherbatov@tarantool.org> (raw) Sometimes field data could have complex document structure. When this structure is consistent across whole document, you are able to create an index by JSON path. Changes in version 6: - key_part(s) path string is not 0-terminated anymore - global formats_epoch pool used to initialize tuple_format:epoch numeric field - more informative error messages - splitted changes to few new additional preparatory commits - got rid off hashtable in vy_stmt_build - prevent traversing field tree twice in vy_stmt_build - inline basic field_by_part with slowpath alternative - simplified tuple_format1_can_store_format2_tuples - splitted tuple_init_field_map to helper routines - moved tests to separate file - large and descriptive comments for many things http://github.com/tarantool/tarantool/tree/kshch/gh-1012-json-indexes https://github.com/tarantool/tarantool/issues/1012 Kirill Shcherbatov (8): box: refactor tuple_validate_raw box: refactor ER_{FIELD_TYPE, ACTION_MISMATCH} box: build path to field string uniformly on error box: introduce JSON Indexes box: introduce has_json_paths flag in templates box: tune tuple_field_raw_by_path for indexed data box: introduce offset_slot cache in key_part box: specify indexes in user-friendly form src/box/alter.cc | 2 +- src/box/errcode.h | 6 +- src/box/index_def.c | 10 +- src/box/key_def.c | 172 +++++++-- src/box/key_def.h | 44 ++- src/box/lua/index.c | 64 ++++ src/box/lua/schema.lua | 22 +- src/box/lua/space.cc | 5 + src/box/memtx_engine.c | 4 + src/box/memtx_rtree.c | 2 +- src/box/sql.c | 1 + src/box/sql/build.c | 1 + src/box/sql/select.c | 3 +- src/box/sql/where.c | 1 + src/box/tuple.c | 35 +- src/box/tuple.h | 34 +- src/box/tuple_compare.cc | 125 +++++-- src/box/tuple_extract_key.cc | 128 +++++-- src/box/tuple_format.c | 666 ++++++++++++++++++++++++++++++----- src/box/tuple_format.h | 111 ++++-- src/box/tuple_hash.cc | 47 ++- src/box/vinyl.c | 4 + src/box/vy_log.c | 11 +- src/box/vy_point_lookup.c | 2 - src/box/vy_stmt.c | 105 +++++- test/box/alter.result | 3 +- test/box/blackhole.result | 6 +- test/box/ddl.result | 6 +- test/box/misc.result | 1 + test/box/rtree_misc.result | 3 +- test/engine/ddl.result | 57 +-- test/engine/json.result | 483 +++++++++++++++++++++++++ test/engine/json.test.lua | 139 ++++++++ test/engine/null.result | 24 +- test/engine/update.result | 6 +- test/vinyl/errinj.result | 3 +- 36 files changed, 1975 insertions(+), 361 deletions(-) create mode 100644 test/engine/json.result create mode 100644 test/engine/json.test.lua -- 2.19.2
next reply other threads:[~2018-12-17 6:52 UTC|newest] Thread overview: 14+ messages / expand[flat|nested] mbox.gz Atom feed top 2018-12-17 6:52 Kirill Shcherbatov [this message] 2018-12-17 6:52 ` [PATCH v6 1/8] box: refactor tuple_validate_raw Kirill Shcherbatov 2018-12-17 6:52 ` [PATCH v6 2/8] box: refactor ER_{FIELD_TYPE, ACTION_MISMATCH} Kirill Shcherbatov 2018-12-17 6:52 ` [PATCH v6 3/8] box: build path to field string uniformly on error Kirill Shcherbatov 2018-12-17 6:52 ` [PATCH v6 4/8] box: introduce JSON Indexes Kirill Shcherbatov 2018-12-17 6:52 ` [PATCH v6 5/8] box: introduce has_json_paths flag in templates Kirill Shcherbatov 2018-12-27 11:51 ` [tarantool-patches] " Konstantin Osipov 2018-12-27 11:52 ` Konstantin Osipov 2018-12-27 11:57 ` [tarantool-patches] " Konstantin Osipov 2018-12-17 6:52 ` [PATCH v6 6/8] box: tune tuple_field_raw_by_path for indexed data Kirill Shcherbatov 2018-12-17 6:52 ` [PATCH v6 7/8] box: introduce offset_slot cache in key_part Kirill Shcherbatov 2018-12-17 6:52 ` [PATCH v6 8/8] box: specify indexes in user-friendly form Kirill Shcherbatov 2018-12-18 20:58 ` Vladimir Davydov 2018-12-18 20:46 ` [PATCH v6 0/8] box: Indexes by JSON path Vladimir Davydov
Reply instructions: You may reply publicly to this message via plain-text email using any one of the following methods: * Save the following mbox file, import it into your mail client, and reply-to-all from there: mbox Avoid top-posting and favor interleaved quoting: https://en.wikipedia.org/wiki/Posting_style#Interleaved_style * Reply using the --to, --cc, and --in-reply-to switches of git-send-email(1): git send-email \ --in-reply-to=cover.1544995259.git.kshcherbatov@tarantool.org \ --to=kshcherbatov@tarantool.org \ --cc=kostja@tarantool.org \ --cc=tarantool-patches@freelists.org \ --cc=vdavydov.dev@gmail.com \ --subject='Re: [PATCH v6 0/8] box: Indexes by JSON path' \ /path/to/YOUR_REPLY https://kernel.org/pub/software/scm/git/docs/git-send-email.html * If your mail client supports setting the In-Reply-To header via mailto: links, try the mailto: link
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox