Tarantool development patches archive
 help / color / mirror / Atom feed
From: Kirill Shcherbatov <kshcherbatov@tarantool.org>
To: tarantool-patches@freelists.org
Cc: "n.pettik@corp.mail.ru" <n.pettik@corp.mail.ru>
Subject: [tarantool-patches] Re: [PATCH v1 0/2] SQL Tests failed on concurrent launch
Date: Mon, 25 Jun 2018 16:39:37 +0300	[thread overview]
Message-ID: <6190b00b-e25c-7789-4179-4d1bbf616a88@tarantool.org> (raw)
In-Reply-To: <cover.1529510053.git.kshcherbatov@tarantool.org>

Ok, let's use cleanup on restart server and squash all changes into one commit.

---
 test/sql/transitive-transactions.result   | 2 ++
 test/sql/transitive-transactions.test.lua | 2 ++
 test/sql/view_delayed_wal.result          | 2 +-
 test/sql/view_delayed_wal.test.lua        | 2 +-
 4 files changed, 6 insertions(+), 2 deletions(-)

diff --git a/test/sql/transitive-transactions.result b/test/sql/transitive-transactions.result
index 37b563a..e617cf1 100644
--- a/test/sql/transitive-transactions.result
+++ b/test/sql/transitive-transactions.result
@@ -118,6 +118,8 @@ box.space.PARENT:select();
   - [2, 2]
 ...
 -- Cleanup
+box.sql.execute('PRAGMA defer_foreign_keys = 0;')
+
 box.sql.execute('DROP TABLE child;');
 ---
 ...
diff --git a/test/sql/transitive-transactions.test.lua b/test/sql/transitive-transactions.test.lua
index 14a1e8c..718f951 100644
--- a/test/sql/transitive-transactions.test.lua
+++ b/test/sql/transitive-transactions.test.lua
@@ -60,6 +60,8 @@ fk_defer();
 box.space.CHILD:select();
 box.space.PARENT:select();
 
+box.sql.execute('PRAGMA defer_foreign_keys = 0;')
+
 -- Cleanup
 box.sql.execute('DROP TABLE child;');
 box.sql.execute('DROP TABLE parent;');
diff --git a/test/sql/view_delayed_wal.result b/test/sql/view_delayed_wal.result
index 50efbee..3b68eda 100644
--- a/test/sql/view_delayed_wal.result
+++ b/test/sql/view_delayed_wal.result
@@ -100,4 +100,4 @@ box.space.V2
 -- Since deletion via Lua doesn't remove entry from
 -- SQL data dictionary we have to restart instance to clean up.
 --
-test_run:cmd('restart server default')
+test_run:cmd('restart server default with cleanup=1')
diff --git a/test/sql/view_delayed_wal.test.lua b/test/sql/view_delayed_wal.test.lua
index 630c52b..cca75ad 100644
--- a/test/sql/view_delayed_wal.test.lua
+++ b/test/sql/view_delayed_wal.test.lua
@@ -42,4 +42,4 @@ box.space.V2
 -- Since deletion via Lua doesn't remove entry from
 -- SQL data dictionary we have to restart instance to clean up.
 --
-test_run:cmd('restart server default')
+test_run:cmd('restart server default with cleanup=1')
-- 
2.7.4

  parent reply	other threads:[~2018-06-25 13:39 UTC|newest]

Thread overview: 9+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-20 15:55 [tarantool-patches] " Kirill Shcherbatov
2018-06-20 15:55 ` [tarantool-patches] [PATCH v1 1/2] sql: finish clean-up in view_delayed_wal test Kirill Shcherbatov
2018-06-21 15:48   ` [tarantool-patches] " n.pettik
2018-06-20 15:55 ` [tarantool-patches] [PATCH v1 2/2] sql: unset pragma in transitive-transactions test Kirill Shcherbatov
2018-06-21 15:45   ` [tarantool-patches] " n.pettik
2018-06-21 16:06     ` Kirill Shcherbatov
2018-06-25 13:39 ` Kirill Shcherbatov [this message]
2018-06-25 13:42   ` [tarantool-patches] Re: [PATCH v1 0/2] SQL Tests failed on concurrent launch n.pettik
2018-06-26  8:01     ` Kirill Yukhin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=6190b00b-e25c-7789-4179-4d1bbf616a88@tarantool.org \
    --to=kshcherbatov@tarantool.org \
    --cc=n.pettik@corp.mail.ru \
    --cc=tarantool-patches@freelists.org \
    --subject='[tarantool-patches] Re: [PATCH v1 0/2] SQL Tests failed on concurrent launch' \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox