[tarantool-patches] [PATCH 0/2] sql: update inexes after table rename

Kirill Yukhin kyukhin at tarantool.org
Mon Aug 20 19:49:43 MSK 2018


First patch in the patchset fixes indexes update, absense of which caused
gh-3613 to fail. After that, another issue was uncovered. It seems that
during comparing of index defs, opts.sql field was ignored, which in turn
leads to ifnorance of updated indexes from xlog. Second patch takes ops.sql
into account while comparing.

Issue: https://github.com/tarantool/tarantool/issues/3613
Branch: https://github.com/tarantool/tarantool/commits/kyukhin/gh-3613-update-idx-afer-alter

Kirill Yukhin (2):
  sql: after table rename properly update indexes
  sql: take sql field in index_opts_cmp

 src/box/index_def.h                          |  8 ++++
 src/box/sql.c                                | 57 +++++++++++++++++++++-------
 src/box/sql/build.c                          |  7 ++--
 src/box/sql/tarantoolInt.h                   | 23 ++++++++++-
 src/box/sql/vdbe.c                           | 26 ++++++++++++-
 test/sql/gh-3613-idx-alter-update-2.result   | 28 ++++++++++++++
 test/sql/gh-3613-idx-alter-update-2.test.lua | 16 ++++++++
 test/sql/gh-3613-idx-alter-update.result     | 38 +++++++++++++++++++
 test/sql/gh-3613-idx-alter-update.test.lua   | 21 ++++++++++
 9 files changed, 203 insertions(+), 21 deletions(-)
 create mode 100644 test/sql/gh-3613-idx-alter-update-2.result
 create mode 100644 test/sql/gh-3613-idx-alter-update-2.test.lua
 create mode 100644 test/sql/gh-3613-idx-alter-update.result
 create mode 100644 test/sql/gh-3613-idx-alter-update.test.lua

-- 
2.16.2





More information about the Tarantool-patches mailing list