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 C726B23012 for ; Thu, 3 May 2018 06:10:12 -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 4z2xZstpLEJp for ; Thu, 3 May 2018 06:10:12 -0400 (EDT) Received: from smtp31.i.mail.ru (smtp31.i.mail.ru [94.100.177.91]) (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 BBD0223007 for ; Thu, 3 May 2018 06:10:10 -0400 (EDT) Received: from [185.6.245.156] (port=25537 helo=msk-wire-v_shpilevoy-v.shpilevoy.mail.msk) by smtp31.i.mail.ru with esmtpa (envelope-from ) id 1fEBBh-0001nu-3M for tarantool-patches@freelists.org; Thu, 03 May 2018 13:10:09 +0300 Subject: [tarantool-patches] Re: [PATCH v4 0/7] sql: refactor SQL Parser structures References: From: Vladislav Shpilevoy Message-ID: Date: Thu, 3 May 2018 13:10:08 +0300 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Language: en-US Content-Transfer-Encoding: 7bit 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 Hello. On 28/04/2018 21:26, Kirill Shcherbatov wrote: > Branch: http://github.com/tarantool/tarantool/tree/gh-3272-no-sql-names > Issue: https://github.com/tarantool/tarantool/issues/3272 1. Please, describe the patchset here. > > Kirill Shcherbatov (7): > sql: fix code style in sqlite3Pragma > sql: remove zName and nColumn from SQL > sql: start using type from space_def > sql: start using collations and is_nullable from space_def > sql: move names to server > sql: start using is_view field from space_def > sql: space_def* instead of Table* in Expr > > src/box/field_def.c | 1 + > src/box/field_def.h | 5 + > src/box/space_def.c | 29 ++-- > src/box/sql.c | 156 +++++++++++++++++---- > src/box/sql.h | 32 +++++ > src/box/sql/alter.c | 54 ++++--- > src/box/sql/analyze.c | 16 ++- > src/box/sql/build.c | 364 ++++++++++++++++++++++++++---------------------- > src/box/sql/delete.c | 25 ++-- > src/box/sql/expr.c | 65 +++++---- > src/box/sql/fkey.c | 51 ++++--- > src/box/sql/hash.c | 5 +- > src/box/sql/insert.c | 88 ++++++------ > src/box/sql/pragma.c | 96 +++++++------ > src/box/sql/prepare.c | 26 ++-- > src/box/sql/resolve.c | 33 +++-- > src/box/sql/select.c | 185 +++++++++++++----------- > src/box/sql/sqliteInt.h | 37 ++--- > src/box/sql/tokenize.c | 3 + > src/box/sql/treeview.c | 2 +- > src/box/sql/trigger.c | 6 +- > src/box/sql/update.c | 52 +++---- > src/box/sql/util.c | 9 -- > src/box/sql/vdbe.c | 2 +- > src/box/sql/vdbeaux.c | 19 ++- > src/box/sql/where.c | 24 ++-- > src/box/sql/wherecode.c | 23 +-- > src/box/sql/whereexpr.c | 8 +- > 28 files changed, 831 insertions(+), 585 deletions(-) >