From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id AA98D26300 for ; Tue, 10 Jul 2018 13:08:17 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id vopAE1Ze_hp6 for ; Tue, 10 Jul 2018 13:08:17 -0400 (EDT) Received: from smtpng2.m.smailru.net (smtpng2.m.smailru.net [94.100.179.3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id 67C0B262E6 for ; Tue, 10 Jul 2018 13:08:17 -0400 (EDT) From: Kirill Shcherbatov Subject: [tarantool-patches] [PATCH v2 0/4] sql: get rid off sqlite3NestedParse Date: Tue, 10 Jul 2018 20:08:07 +0300 Message-Id: Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-help: List-unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-subscribe: List-owner: List-post: List-archive: To: tarantool-patches@freelists.org Cc: korablev@tarantool.org, Kirill Shcherbatov As we are going to implement parser as separate library, we should get rid off sqlite3NestedParse calls. Last usages in statistics update could be trivially rewrited to do not start parsing. We manually build AST structures simmilar to parser did and manually call sql_table_delete_from. Finilly, I've removed all sqlite3NestedParse mentionings. Also slightly refactored OP_Open{Write,Read} opcodes and vdbe_emit_open_cursor function to use index_id instead of tnum. Branch: http://github.com/tarantool/tarantool/tree/kshch/gh-3496-no-nested-parse Issue: https://github.com/tarantool/tarantool/issues/3496 Kirill Shcherbatov (3): sql: get rid off sqlite3NestedParse in clean stats sql: remove usless sqlite3NestedParse function sql: refactor vdbe_emit_open_cursor calls Vladislav Shpilevoy (1): sql: remove OP_LoadPtr src/box/sql/analyze.c | 110 +++++++++++--------------- src/box/sql/build.c | 202 ++++++++++++++++++------------------------------ src/box/sql/delete.c | 32 ++------ src/box/sql/expr.c | 5 +- src/box/sql/fkey.c | 3 +- src/box/sql/insert.c | 120 ++++++++++++++-------------- src/box/sql/parse.y | 12 +-- src/box/sql/pragma.c | 36 +++++---- src/box/sql/select.c | 4 +- src/box/sql/sqliteInt.h | 39 +++++----- src/box/sql/tokenize.c | 2 +- src/box/sql/trigger.c | 45 ++++------- src/box/sql/update.c | 15 ++-- src/box/sql/vdbe.c | 73 +++-------------- src/box/sql/where.c | 10 ++- 15 files changed, 272 insertions(+), 436 deletions(-) -- 2.7.4