[tarantool-patches] Re: [PATCH v5 0/3] sql: refactor SQL Parser structures

[Vladislav Shpilevoy]

On 11/05/2018 11:49, Kirill Shcherbatov wrote:
> Branch: http://github.com/tarantool/tarantool/tree/gh-3272-no-sql-names
> Issue: https://github.com/tarantool/tarantool/issues/3272

Hello. Describe the changes here, and why they are needed.

> 
> Kirill Shcherbatov (3):
>    sql: fix code style in sqlite3Pragma
>    sql: remove SQL fields from Table and Column
>    sql: space_def* instead of Table* in Expr
> 
>   src/box/field_def.c     |   1 +
>   src/box/field_def.h     |  20 +++
>   src/box/space_def.c     |  29 ++--
>   src/box/sql.c           | 134 +++++++++++++----
>   src/box/sql.h           |  33 +++++
>   src/box/sql/alter.c     |  55 ++++---
>   src/box/sql/analyze.c   |  16 +-
>   src/box/sql/build.c     | 378 ++++++++++++++++++++++++++----------------------
>   src/box/sql/delete.c    |  25 ++--
>   src/box/sql/expr.c      | 119 ++++++++-------
>   src/box/sql/fkey.c      |  51 +++----
>   src/box/sql/hash.c      |   5 +-
>   src/box/sql/hash.h      |   2 +-
>   src/box/sql/insert.c    | 106 ++++++++------
>   src/box/sql/pragma.c    |  96 ++++++------
>   src/box/sql/prepare.c   |  45 +++---
>   src/box/sql/resolve.c   |  32 ++--
>   src/box/sql/select.c    | 183 +++++++++++++----------
>   src/box/sql/sqliteInt.h |  78 +++++-----
>   src/box/sql/tokenize.c  |   5 +-
>   src/box/sql/treeview.c  |   2 +-
>   src/box/sql/trigger.c   |   7 +-
>   src/box/sql/update.c    |  58 ++++----
>   src/box/sql/util.c      |   9 --
>   src/box/sql/vdbe.c      |   2 +-
>   src/box/sql/vdbeaux.c   |  25 +---
>   src/box/sql/where.c     |  25 ++--
>   src/box/sql/wherecode.c |  23 +--
>   src/box/sql/whereexpr.c |   8 +-
>   29 files changed, 899 insertions(+), 673 deletions(-)
>