[tarantool-patches] Re: [PATCH v2 3/3] sql: enable multi-engine tests for SQL

[Kirill Shcherbatov]

> Could we disable them for vinyl somehow and run only once?
> Some of them take long time to execute.
This way:

+++ b/test/sql-tap/engine.cfg
@@ -1,4 +1,7 @@
 {
+    "analyze9.test.lua": {
+        "memtx": {"engine": "memtx"}
+    },
     "*": {
         "memtx": {"engine": "memtx"},
         "vinyl": {"engine": "vinyl"}

+++ b/test/sql-tap/analyze9.test.lua

-        pragma sql_default_engine='memtx';

> Lets put newline at the end of file.
Ok, fixed on merge.

> Please, use types for columns: we are going to introduce static types
> in SQL and make them mandatory.

-box.sql.execute("CREATE TABLE t1_vinyl(a primary key,b,c);")
-box.sql.execute("CREATE TABLE t2_vinyl(a primary key,b,c);")
+box.sql.execute("CREATE TABLE t1_vinyl(a INT PRIMARY KEY, b INT, c INT);")
+box.sql.execute("CREATE TABLE t2_vinyl(a INT PRIMARY KEY, b INT, c INT);")

-box.sql.execute("CREATE TABLE t3_memtx(a primary key,b,c);")
+box.sql.execute("CREATE TABLE t3_memtx(a INT PRIMARY KEY, b INT, c INT);")