From: Kirill Shcherbatov <kshcherbatov@tarantool.org>
To: tarantool-patches@freelists.org
Cc: Vladislav Shpilevoy <v.shpilevoy@tarantool.org>
Subject: [tarantool-patches] Re: [PATCH v6 0/4] sql: moved Checks to server
Date: Wed, 16 May 2018 16:13:33 +0300 [thread overview]
Message-ID: <be145722-b132-b049-3102-510fab9d5132@tarantool.org> (raw)
In-Reply-To: <1e67c524-f635-15ce-7a89-b3341c4caa9b@tarantool.org>
> Hello. Thanks for the patchset! Please, describe what it does, why and how.
Refactored SQL Column and Table structures to use space_def instead of native fields.
All temporal memory is allocated on region to prevent memory leaks and easely manage memory
with frequent insertions on SQL expression parsing.
Introduced space_def pointer in Expr instead of Table pointer.
The ResolveSelfReference function requires Table pointer to refer to, but it doesn't exist in our scenarios yet.
With correctly-initialized consistent space_def structure this would no problem.
This preliminary tasks made able to move SQL Checks in Server code.
For checks ExprList transfering to server data is packed in msgpack as array of maps:
[{"expr_str": "x < y", "name" : "FIRST"}, ..]
Introduced checks_array_decode aimed to unpack this complex package.
Introduced sql_update_space_def_reference to update space references as space_def pointer
changes over the i.e. resolved checks refer released memory.
next prev parent reply other threads:[~2018-05-16 13:13 UTC|newest]
Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top
2018-05-15 17:03 [tarantool-patches] " Kirill Shcherbatov
2018-05-15 17:03 ` [tarantool-patches] [PATCH v6 1/4] sql: fix code style in sqlite3Pragma Kirill Shcherbatov
2018-05-15 17:03 ` [tarantool-patches] [PATCH v6 2/4] sql: remove SQL fields from Table and Column Kirill Shcherbatov
2018-05-17 17:25 ` [tarantool-patches] " n.pettik
2018-05-18 15:35 ` Kirill Shcherbatov
2018-05-18 17:24 ` n.pettik
2018-05-18 19:45 ` Kirill Shcherbatov
2018-05-18 20:13 ` n.pettik
2018-05-15 17:03 ` [tarantool-patches] [PATCH v6 3/4] sql: space_def* instead of Table* in Expr Kirill Shcherbatov
2018-05-16 12:33 ` [tarantool-patches] " Vladislav Shpilevoy
2018-05-16 13:10 ` Kirill Shcherbatov
2018-05-16 13:11 ` Vladislav Shpilevoy
[not found] ` <26E4269B-2BCB-42C3-8216-D51E290E4723@corp.mail.ru>
2018-05-18 15:26 ` Kirill Shcherbatov
2018-05-18 17:04 ` n.pettik
2018-05-21 12:48 ` [tarantool-patches] " Nikita Pettik
2018-05-15 17:03 ` [tarantool-patches] [PATCH v6 4/4] sql: remove Checks to server Kirill Shcherbatov
2018-05-16 17:59 ` [tarantool-patches] " Vladislav Shpilevoy
2018-05-16 11:52 ` [tarantool-patches] Re: [PATCH v6 0/4] sql: moved " Vladislav Shpilevoy
2018-05-16 13:13 ` Kirill Shcherbatov [this message]
2018-05-23 5:19 ` Kirill Yukhin
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=be145722-b132-b049-3102-510fab9d5132@tarantool.org \
--to=kshcherbatov@tarantool.org \
--cc=tarantool-patches@freelists.org \
--cc=v.shpilevoy@tarantool.org \
--subject='[tarantool-patches] Re: [PATCH v6 0/4] sql: moved Checks to server' \
/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