[tarantool-patches] Re: [PATCH v4 2/7] sql: remove zName and nColumn from SQL

[Vladislav Shpilevoy]

Hello. Same as in the previous letter.

On 28/04/2018 21:26, Kirill Shcherbatov wrote:
> 1. Removed zName from SQL Column.
> 2. Removed zColumns from SQL Table.
> 3. Refactored Parser to use def_expression directly.
> 4. Introduced sql_table_def_rebuild intended for collect
> fragmented with sql_field_retrieve space_def into memory
> located in one allocation.
> 
> Part of #3272.
> ---
>   src/box/space_def.c     |  29 ++++---
>   src/box/sql.c           |  79 +++++++++++++++--
>   src/box/sql.h           |  32 +++++++
>   src/box/sql/alter.c     |  32 ++++---
>   src/box/sql/analyze.c   |   5 +-
>   src/box/sql/build.c     | 221 +++++++++++++++++++++++-------------------------
>   src/box/sql/delete.c    |   6 +-
>   src/box/sql/expr.c      |  11 +--
>   src/box/sql/fkey.c      |  20 ++---
>   src/box/sql/insert.c    |  55 ++++++------
>   src/box/sql/pragma.c    |  24 ++++--
>   src/box/sql/prepare.c   |  26 +++---
>   src/box/sql/resolve.c   |  16 ++--
>   src/box/sql/select.c    | 112 ++++++++++++++----------
>   src/box/sql/sqliteInt.h |   6 +-
>   src/box/sql/tokenize.c  |   3 +
>   src/box/sql/trigger.c   |   2 +
>   src/box/sql/update.c    |  29 ++++---
>   src/box/sql/where.c     |   6 +-
>   src/box/sql/wherecode.c |   2 +-
>   src/box/sql/whereexpr.c |   4 +-
>   21 files changed, 433 insertions(+), 287 deletions(-)
>