[patches] [PATCH V2 0/9] SQLite backend removal

Bulat Niatshin niatshin at tarantool.org
Tue Feb 27 16:54:45 MSK 2018


Branch:
https://github.com/tarantool/tarantool/tree/bn/gh-3119-backend-removal
Issue:
https://github.com/tarantool/tarantool/issues/3119

Brief overview:
SQLite backend is a appendix element in our code base,
which components  like memory journal,
cursor payload, sqlite authorization needs to be
partly removed, partly refactored.
In this patch I've deleted and
refactored it (whole follow-up list you can find in
issue from above).

Changes in V2:
- Fixed comments as per review by Nikita
- Splitted last commit with removing OP_Transaction
  and sqlite authorization into two separate.
- Removed undeleted deprecated code.
- Removed struct CursorPayload.
- Made some commit messages more informative.


Bulat Niatshin (9):
  sql: remove OMIT_AUTOINCREMENT ifndef
  sql: remove P4_TABLE type from 4th vdbe arg
  sql: remove OMIT_DEPRECATED code under define
  sql: remove struct Db
  sql: refactor struct Schema
  sql: remove journal_mode
  sql: delete CursorPayload structure
  sql: remove sqlite authorization
  sql: remove OP_Transaction

 extra/mkopcodeh.tcl        |   2 +-
 src/box/sql.c              |   8 +-
 src/box/sql/CMakeLists.txt |   2 -
 src/box/sql/alter.c        |  27 +--
 src/box/sql/analyze.c      |   8 +-
 src/box/sql/attach.c       |   2 +-
 src/box/sql/build.c        | 218 +++------------------
 src/box/sql/callback.c     |  13 +-
 src/box/sql/cursor.h       |  19 --
 src/box/sql/delete.c       |  20 +-
 src/box/sql/fkey.c         |  19 +-
 src/box/sql/func.c         |  12 --
 src/box/sql/global.c       |  14 +-
 src/box/sql/insert.c       |   3 -
 src/box/sql/main.c         | 100 +---------
 src/box/sql/malloc.c       |  17 --
 src/box/sql/memjournal.c   | 473 ---------------------------------------------
 src/box/sql/opcodes.c      | 216 ++++++++++-----------
 src/box/sql/opcodes.h      | 248 ++++++++++++------------
 src/box/sql/os.c           |   2 +-
 src/box/sql/os_unix.c      | 127 ++----------
 src/box/sql/pragma.c       |  26 +--
 src/box/sql/prepare.c      |  42 +---
 src/box/sql/resolve.c      |  24 ---
 src/box/sql/select.c       |  13 --
 src/box/sql/sqlite3.h      | 368 +----------------------------------
 src/box/sql/sqliteInt.h    | 135 +------------
 src/box/sql/sqliteLimit.h  |   8 -
 src/box/sql/status.c       |   2 +-
 src/box/sql/tarantoolInt.h |   6 +-
 src/box/sql/trigger.c      |  40 +---
 src/box/sql/update.c       |  30 +--
 src/box/sql/vdbe.c         | 234 +++-------------------
 src/box/sql/vdbe.h         |   8 +-
 src/box/sql/vdbeapi.c      |  66 -------
 src/box/sql/vdbeaux.c      |  14 +-
 src/box/sql/vdbesort.c     |  12 +-
 37 files changed, 357 insertions(+), 2221 deletions(-)
 delete mode 100644 src/box/sql/memjournal.c

-- 
2.14.1




More information about the Tarantool-patches mailing list