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 634FA27570 for ; Mon, 20 Aug 2018 12:49:54 -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 rmbO_tHY0WFe for ; Mon, 20 Aug 2018 12:49:54 -0400 (EDT) Received: from smtp62.i.mail.ru (smtp62.i.mail.ru [217.69.128.42]) (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 B79C3273CE for ; Mon, 20 Aug 2018 12:49:53 -0400 (EDT) From: Kirill Yukhin Subject: [tarantool-patches] [PATCH 0/2] sql: update inexes after table rename Date: Mon, 20 Aug 2018 19:49:43 +0300 Message-Id: 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: korablev@tarantool.org Cc: tarantool-patches@freelists.org, Kirill Yukhin 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