From: imeevma@tarantool.org To: v.shpilevoy@tarantool.org Cc: tarantool-patches@freelists.org, tarantool-patches@dev.tarantool.org Subject: [Tarantool-patches] [PATCH v1 5/5] sql: replace control pragmas Date: Thu, 17 Oct 2019 17:40:20 +0300 [thread overview] Message-ID: <40c780065c6746ac8891732dc5bb0e16938421cc.1571322776.git.imeevma@gmail.com> (raw) In-Reply-To: <cover.1571322776.git.imeevma@gmail.com> This patch removes pragmas that modify SQL settings. Most of these pragmas have been replaced by SET. But there are two pragmas that have been completely removed: 'short_column_names' and 'count_changes'. Closes #4511 --- src/box/sql.c | 3 +- src/box/sql/delete.c | 26 --- src/box/sql/insert.c | 35 +--- src/box/sql/parse.y | 3 - src/box/sql/pragma.c | 203 +-------------------- src/box/sql/pragma.h | 150 --------------- src/box/sql/select.c | 16 +- src/box/sql/sqlInt.h | 6 - src/box/sql/update.c | 26 --- src/box/sql/vdbe.c | 26 +-- test/sql-tap/analyze4.test.lua | 4 +- test/sql-tap/autoinc.test.lua | 2 +- test/sql-tap/colname.test.lua | 37 ++-- test/sql-tap/fkey2.test.lua | 4 +- test/sql-tap/gh2548-select-compound-limit.test.lua | 8 +- test/sql-tap/lua/sqltester.lua | 2 +- test/sql-tap/misc1.test.lua | 2 +- test/sql-tap/pragma.test.lua | 135 +------------- test/sql-tap/select1.test.lua | 35 ++-- test/sql-tap/tkt3731.test.lua | 2 +- test/sql-tap/trigger2.test.lua | 2 +- test/sql-tap/triggerC.test.lua | 51 +----- test/sql-tap/update.test.lua | 17 +- test/sql-tap/whereA.test.lua | 24 +-- test/sql/check-clear-ephemeral.result | 4 +- test/sql/check-clear-ephemeral.test.lua | 4 +- test/sql/checks.result | 2 +- test/sql/checks.test.lua | 2 +- test/sql/clear.result | 4 +- test/sql/clear.test.lua | 4 +- test/sql/collation.result | 2 +- test/sql/collation.test.lua | 2 +- test/sql/ddl.result | 2 +- test/sql/ddl.test.lua | 2 +- test/sql/delete-multiple-idx.result | 4 +- test/sql/delete-multiple-idx.test.lua | 4 +- test/sql/delete.result | 4 +- test/sql/delete.test.lua | 4 +- test/sql/drop-index.result | 4 +- test/sql/drop-index.test.lua | 4 +- test/sql/drop-table.result | 4 +- test/sql/drop-table.test.lua | 4 +- test/sql/engine.result | 6 +- test/sql/engine.test.lua | 6 +- test/sql/errinj.result | 2 +- test/sql/errinj.test.lua | 2 +- test/sql/func-recreate.result | 2 +- test/sql/func-recreate.test.lua | 2 +- test/sql/gh-2362-select-access-rights.result | 2 +- test/sql/gh-2362-select-access-rights.test.lua | 2 +- test/sql/gh-2929-primary-key.result | 2 +- test/sql/gh-2929-primary-key.test.lua | 2 +- test/sql/gh-2981-check-autoinc.result | 2 +- test/sql/gh-2981-check-autoinc.test.lua | 2 +- test/sql/gh-3199-no-mem-leaks.result | 2 +- test/sql/gh-3199-no-mem-leaks.test.lua | 2 +- test/sql/gh-3613-idx-alter-update-2.result | 2 +- test/sql/gh-3613-idx-alter-update-2.test.lua | 2 +- test/sql/gh-3613-idx-alter-update.result | 2 +- test/sql/gh-3613-idx-alter-update.test.lua | 2 +- test/sql/gh-3888-values-blob-assert.result | 2 +- test/sql/gh-3888-values-blob-assert.test.lua | 2 +- test/sql/gh2141-delete-trigger-drop-table.result | 2 +- test/sql/gh2141-delete-trigger-drop-table.test.lua | 2 +- test/sql/gh2251-multiple-update.result | 2 +- test/sql/gh2251-multiple-update.test.lua | 2 +- test/sql/gh2483-remote-persistency-check.result | 2 +- test/sql/gh2483-remote-persistency-check.test.lua | 2 +- .../gh2808-inline-unique-persistency-check.result | 2 +- ...gh2808-inline-unique-persistency-check.test.lua | 2 +- test/sql/icu-upper-lower.result | 2 +- test/sql/icu-upper-lower.test.lua | 2 +- test/sql/insert-unique.result | 4 +- test/sql/insert-unique.test.lua | 4 +- test/sql/integer-overflow.result | 2 +- test/sql/integer-overflow.test.lua | 2 +- test/sql/iproto.result | 33 +--- test/sql/iproto.test.lua | 10 +- test/sql/max-on-index.result | 4 +- test/sql/max-on-index.test.lua | 4 +- test/sql/message-func-indexes.result | 2 +- test/sql/message-func-indexes.test.lua | 2 +- test/sql/misc.result | 2 +- test/sql/misc.test.lua | 2 +- test/sql/no-pk-space.result | 2 +- test/sql/no-pk-space.test.lua | 2 +- test/sql/on-conflict.result | 2 +- test/sql/on-conflict.test.lua | 2 +- test/sql/persistency.result | 2 +- test/sql/persistency.test.lua | 2 +- test/sql/row-count.result | 10 +- test/sql/row-count.test.lua | 4 +- test/sql/savepoints.result | 2 +- test/sql/savepoints.test.lua | 2 +- test/sql/select-null.result | 4 +- test/sql/select-null.test.lua | 4 +- test/sql/sql-debug.result | 50 ----- test/sql/sql-debug.test.lua | 15 -- test/sql/sql-statN-index-drop.result | 2 +- test/sql/sql-statN-index-drop.test.lua | 2 +- test/sql/tokenizer.result | 2 +- test/sql/tokenizer.test.lua | 2 +- test/sql/transition.result | 2 +- test/sql/transition.test.lua | 2 +- test/sql/transitive-transactions.result | 8 +- test/sql/transitive-transactions.test.lua | 8 +- test/sql/triggers.result | 14 +- test/sql/triggers.test.lua | 14 +- test/sql/update-with-nested-select.result | 4 +- test/sql/update-with-nested-select.test.lua | 4 +- test/sql/upgrade.result | 2 +- test/sql/upgrade.test.lua | 2 +- test/sql/view.result | 2 +- test/sql/view.test.lua | 2 +- test/sql/view_delayed_wal.result | 2 +- test/sql/view_delayed_wal.test.lua | 2 +- test/sql/vinyl-opts.result | 2 +- test/sql/vinyl-opts.test.lua | 2 +- 118 files changed, 207 insertions(+), 994 deletions(-) diff --git a/src/box/sql.c b/src/box/sql.c index 0c73caf..446b0ab 100644 --- a/src/box/sql.c +++ b/src/box/sql.c @@ -59,8 +59,7 @@ static sql *db = NULL; static const char nil_key[] = { 0x90 }; /* Empty MsgPack array. */ -static const uint32_t default_sql_flags = SQL_ShortColNames - | SQL_EnableTrigger +static const uint32_t default_sql_flags = SQL_EnableTrigger | SQL_AutoIndex | SQL_RecTriggers; diff --git a/src/box/sql/delete.c b/src/box/sql/delete.c index 2f73d80..8d9ae4f 100644 --- a/src/box/sql/delete.c +++ b/src/box/sql/delete.c @@ -191,15 +191,6 @@ sql_table_delete_from(struct Parse *parse, struct SrcList *tab_list, tab_cursor); } - /* Initialize the counter of the number of rows deleted, - * if we are counting rows. - */ - int reg_count = -1; - uint32_t sql_flags = parse->sql_flags; - if ((sql_flags & SQL_CountRows) != 0) { - reg_count = ++parse->nMem; - sqlVdbeAddOp2(v, OP_Integer, 0, reg_count); - } /* Special case: A DELETE without a WHERE clause deletes * everything. It is easier just to erase the whole table. */ @@ -284,12 +275,6 @@ sql_table_delete_from(struct Parse *parse, struct SrcList *tab_list, */ /* assert(is_complex || one_pass != ONEPASS_OFF); */ - /* Keep track of the number of rows to be - * deleted. - */ - if ((sql_flags & SQL_CountRows) != 0) - sqlVdbeAddOp2(v, OP_AddImm, reg_count, 1); - /* Extract the primary key for the current row */ if (!is_view) { struct key_part_def *part = pk_info->parts; @@ -413,17 +398,6 @@ sql_table_delete_from(struct Parse *parse, struct SrcList *tab_list, } } - /* Return the number of rows that were deleted. */ - if ((sql_flags & SQL_CountRows) != 0 && - parse->triggered_space != NULL) { - sqlVdbeAddOp2(v, OP_ResultRow, reg_count, 1); - sqlVdbeSetNumCols(v, 1); - sqlVdbeSetColName(v, 0, COLNAME_NAME, "rows deleted", - SQL_STATIC); - sqlVdbeSetColName(v, 0, COLNAME_DECLTYPE, "INTEGER", - SQL_STATIC); - } - delete_from_cleanup: sqlSrcListDelete(db, tab_list); sql_expr_delete(db, where, false); diff --git a/src/box/sql/insert.c b/src/box/sql/insert.c index 42b8391..43a0de5 100644 --- a/src/box/sql/insert.c +++ b/src/box/sql/insert.c @@ -251,7 +251,6 @@ sqlInsert(Parse * pParse, /* Parser context */ /* Register allocations */ int regFromSelect = 0; /* Base register for data coming from SELECT */ - int regRowCount = 0; /* Memory cell used for the row counter */ int regIns; /* Block of regs holding data being inserted */ int regTupleid; /* registers holding insert tupleid */ int regData; /* register holding first column to insert */ @@ -334,7 +333,7 @@ sqlInsert(Parse * pParse, /* Parser context */ xferOptimization(pParse, space, pSelect, on_error)) { assert(trigger == NULL); assert(pList == 0); - goto insert_end; + goto insert_cleanup; } /* @@ -512,12 +511,6 @@ sqlInsert(Parse * pParse, /* Parser context */ goto insert_cleanup; } - /* Initialize the count of rows to be inserted - */ - if ((pParse->sql_flags & SQL_CountRows) != 0) { - regRowCount = ++pParse->nMem; - sqlVdbeAddOp2(v, OP_Integer, 0, regRowCount); - } /* This is the top of the main insertion loop */ if (useTempTable) { /* This block codes the top of loop only. The complete loop is the @@ -745,12 +738,6 @@ sqlInsert(Parse * pParse, /* Parser context */ on_error, autoinc_reg); } - /* Update the count of rows that are inserted - */ - if ((pParse->sql_flags & SQL_CountRows) != 0) { - sqlVdbeAddOp2(v, OP_AddImm, regRowCount, 1); - } - if (trigger != NULL) { /* Code AFTER triggers */ vdbe_code_row_trigger(pParse, trigger, TK_INSERT, 0, @@ -773,23 +760,6 @@ sqlInsert(Parse * pParse, /* Parser context */ sqlVdbeJumpHere(v, addrInsTop); } - insert_end: - - /* Return the number of rows inserted. */ - if ((pParse->sql_flags & SQL_CountRows) != 0 && - pParse->triggered_space == NULL) { - sqlVdbeAddOp2(v, OP_ResultRow, regRowCount, 1); - sqlVdbeSetNumCols(v, 1); - const char *column_name; - if (on_error == ON_CONFLICT_ACTION_REPLACE) - column_name = "rows replaced"; - else - column_name = "rows inserted"; - sqlVdbeSetColName(v, 0, COLNAME_NAME, column_name, SQL_STATIC); - sqlVdbeSetColName(v, 0, COLNAME_DECLTYPE, "INTEGER", - SQL_STATIC); - } - insert_cleanup: sqlSrcListDelete(db, pTabList); sql_expr_list_delete(db, pList); @@ -1182,9 +1152,6 @@ xferOptimization(Parse * pParse, /* Parser context */ */ if (!rlist_empty(&dest->child_fk_constraint)) return 0; - if ((pParse->sql_flags & SQL_CountRows) != 0) { - return 0; /* xfer opt does not play well with PRAGMA count_changes */ - } /* If we get this far, it means that the xfer optimization is at * least a possibility, though it might only work if the destination diff --git a/src/box/sql/parse.y b/src/box/sql/parse.y index 7bea68d..41ea390 100644 --- a/src/box/sql/parse.y +++ b/src/box/sql/parse.y @@ -1563,9 +1563,6 @@ cmd ::= PRAGMA nm(X) LP minus_num(Y) RP. { cmd ::= PRAGMA nm(X) LP nm(Z) DOT nm(Y) RP. { sqlPragma(pParse,&X,&Y,&Z,0); } -cmd ::= PRAGMA . { - sqlPragma(pParse, 0,0,0,0); -} nmnum(A) ::= plus_num(A). nmnum(A) ::= STRING(A). diff --git a/src/box/sql/pragma.c b/src/box/sql/pragma.c index 5bf24ef..0eaa2ca 100644 --- a/src/box/sql/pragma.c +++ b/src/box/sql/pragma.c @@ -62,56 +62,6 @@ #include "pragma.h" #include "tarantoolInt.h" -/* - * Interpret the given string as a safety level. Return 0 for OFF, - * 1 for ON or NORMAL, 2 for FULL, and 3 for EXTRA. Return 1 for an empty or - * unrecognized string argument. The FULL and EXTRA option is disallowed - * if the omitFull parameter it 1. - * - * Note that the values returned are one less that the values that - * should be passed into sqlBtreeSetSafetyLevel(). The is done - * to support legacy SQL code. The safety level used to be boolean - * and older scripts may have used numbers 0 for OFF and 1 for ON. - */ -static u8 -getSafetyLevel(const char *z, int omitFull, u8 dflt) -{ - /* 123456789 123456789 123 */ - static const char zText[] = "onoffalseyestruextrafull"; - static const u8 iOffset[] = { 0, 1, 2, 4, 9, 12, 15, 20 }; - static const u8 iLength[] = { 2, 2, 3, 5, 3, 4, 5, 4 }; - static const u8 iValue[] = { 1, 0, 0, 0, 1, 1, 3, 2 }; - /* on no off false yes true extra full */ - int i, n; - if (sqlIsdigit(*z)) { - return (u8) sqlAtoi(z); - } - n = sqlStrlen30(z); - for (i = 0; i < ArraySize(iLength); i++) { - if (iLength[i] == n - && sqlStrNICmp(&zText[iOffset[i]], z, n) == 0 - && (!omitFull || iValue[i] <= 1) - ) { - return iValue[i]; - } - } - return dflt; -} - -/* - * Interpret the given string as a boolean value. - */ -u8 -sqlGetBoolean(const char *z, u8 dflt) -{ - return getSafetyLevel(z, 1, dflt) != 0; -} - -/* The sqlGetBoolean() function is used by other modules but the - * remainder of this file is specific to PRAGMA processing. So omit - * the rest of the file if PRAGMAs are omitted from the build. - */ - /** Set result column names and types for a pragma. */ static void vdbe_set_pragma_result_columns(struct Vdbe *v, const struct PragmaName *pragma) @@ -128,17 +78,6 @@ vdbe_set_pragma_result_columns(struct Vdbe *v, const struct PragmaName *pragma) } /* - * Generate code to return a single integer value. - */ -static void -returnSingleInt(Vdbe * v, i64 value) -{ - sqlVdbeAddOp4Dup8(v, OP_Int64, 0, 1, 0, (const u8 *)&value, - value < 0 ? P4_INT64 : P4_UINT64); - sqlVdbeAddOp2(v, OP_ResultRow, 1, 1); -} - -/* * Locate a pragma in the aPragmaName[] array. */ static const PragmaName * @@ -161,53 +100,6 @@ pragmaLocate(const char *zName) return lwr > upr ? 0 : &aPragmaName[mid]; } -static void -vdbe_emit_pragma_status(struct Parse *parse) -{ - struct Vdbe *v = sqlGetVdbe(parse); - struct session *user_session = current_session(); - - sqlVdbeSetNumCols(v, 2); - sqlVdbeSetColName(v, 0, COLNAME_NAME, "pragma_name", SQL_STATIC); - sqlVdbeSetColName(v, 0, COLNAME_DECLTYPE, "TEXT", SQL_STATIC); - sqlVdbeSetColName(v, 1, COLNAME_NAME, "pragma_value", SQL_STATIC); - sqlVdbeSetColName(v, 1, COLNAME_DECLTYPE, "INTEGER", SQL_STATIC); - - parse->nMem = 2; - for (int i = 0; i < ArraySize(aPragmaName); ++i) { - if (aPragmaName[i].ePragTyp != PragTyp_FLAG) - continue; - sqlVdbeAddOp4(v, OP_String8, 0, 1, 0, aPragmaName[i].zName, 0); - int val = (user_session->sql_flags & aPragmaName[i].iArg) != 0; - sqlVdbeAddOp2(v, OP_Integer, val, 2); - sqlVdbeAddOp2(v, OP_ResultRow, 1, 2); - } -} - -/** - * Set tarantool backend default engine for SQL interface. - * @param engine_name to set default. - * @retval -1 on error. - * @retval 0 on success. - */ -static int -sql_default_engine_set(const char *engine_name) -{ - if (engine_name == NULL) { - diag_set(ClientError, ER_ILLEGAL_PARAMS, - "'sql_default_engine' was not specified"); - return -1; - } - enum sql_storage_engine engine = - STR2ENUM(sql_storage_engine, engine_name); - if (engine == sql_storage_engine_MAX) { - diag_set(ClientError, ER_NO_SUCH_ENGINE, engine_name); - return -1; - } - current_session()->sql_default_engine = engine; - return 0; -} - /** * This function handles PRAGMA TABLE_INFO(<table>). * @@ -373,22 +265,6 @@ sql_pragma_index_list(struct Parse *parse, const char *tbl_name) } /* - * @brief Check whether the specified token is a string or ID. - * @param token - token to be examined - * @return true - if the token value is enclosed into quotes (') - * @return false in other cases - * The empty value is considered to be a string. - */ -static bool -token_is_string(const struct Token* token) -{ - if (!token || token->n == 0) - return true; - return token->n >= 2 && token->z[0] == '\'' && - token->z[token->n - 1] == '\''; -} - -/* * Process a pragma statement. * * Pragmas are of this form: @@ -416,17 +292,12 @@ sqlPragma(Parse * pParse, Token * pId, /* First part of [schema.]id field */ sql *db = pParse->db; /* The database connection */ Vdbe *v = sqlGetVdbe(pParse); /* Prepared statement */ const PragmaName *pPragma; /* The pragma */ - struct session *user_session = current_session(); if (v == 0) return; sqlVdbeRunOnlyOnce(v); pParse->nMem = 2; - if (pId == NULL) { - vdbe_emit_pragma_status(pParse); - return; - } zLeft = sql_name_from_token(db, pId); if (zLeft == NULL) { pParse->is_aborted = true; @@ -456,38 +327,10 @@ sqlPragma(Parse * pParse, Token * pId, /* First part of [schema.]id field */ goto pragma_out; } /* Register the result column names for pragmas that return results */ - if ((pPragma->mPragFlg & PragFlg_NoColumns) == 0 && - ((pPragma->mPragFlg & PragFlg_NoColumns1) == 0 || zRight == NULL)) - vdbe_set_pragma_result_columns(v, pPragma); + vdbe_set_pragma_result_columns(v, pPragma); /* Jump to the appropriate pragma handler */ switch (pPragma->ePragTyp) { - case PragTyp_FLAG:{ - if (zRight == NULL) { - vdbe_set_pragma_result_columns(v, pPragma); - returnSingleInt(v, (user_session->sql_flags & - pPragma->iArg) != 0); - } else { - /* Mask of bits to set or clear. */ - int mask = pPragma->iArg; - bool is_pragma_set = sqlGetBoolean(zRight, 0); - - if (is_pragma_set) - user_session->sql_flags |= mask; - else - user_session->sql_flags &= ~mask; -#if defined(SQL_DEBUG) - if (mask == PARSER_TRACE_FLAG) { - if (is_pragma_set) - sqlParserTrace(stdout, "parser: "); - else - sqlParserTrace(0, 0); - } -#endif - } - break; - } - case PragTyp_TABLE_INFO: sql_pragma_table_info(pParse, zRight); break; @@ -565,54 +408,10 @@ sqlPragma(Parse * pParse, Token * pId, /* First part of [schema.]id field */ break; } - case PragTyp_DEFAULT_ENGINE: { - if (!token_is_string(pValue)) { - diag_set(ClientError, ER_ILLEGAL_PARAMS, - "string value is expected"); - pParse->is_aborted = true; - goto pragma_out; - } - if (zRight == NULL) { - const char *engine_name = - sql_storage_engine_strs[current_session()-> - sql_default_engine]; - sqlVdbeLoadString(v, 1, engine_name); - sqlVdbeAddOp2(v, OP_ResultRow, 1, 1); - } else { - if (sql_default_engine_set(zRight) != 0) { - pParse->is_aborted = true; - goto pragma_out; - } - sqlVdbeAddOp0(v, OP_Expire); - } - break; - } - - case PragTyp_COMPOUND_SELECT_LIMIT: { - if (zRight != NULL) { - sql_limit(db, SQL_LIMIT_COMPOUND_SELECT, - sqlAtoi(zRight)); - } - int retval = - sql_limit(db, SQL_LIMIT_COMPOUND_SELECT, -1); - returnSingleInt(v, retval); - break; - } - default: unreachable(); } /* End of the PRAGMA switch */ - /* The following block is a no-op unless SQL_DEBUG is - * defined. Its only * purpose is to execute assert() - * statements to verify that if the * PragFlg_NoColumns1 flag - * is set and the caller specified an argument * to the PRAGMA, - * the implementation has not added any OP_ResultRow * - * instructions to the VM. - */ - if ((pPragma->mPragFlg & PragFlg_NoColumns1) && zRight) { - sqlVdbeVerifyNoResultRow(v); - } pragma_out: sqlDbFree(db, zLeft); sqlDbFree(db, zRight); diff --git a/src/box/sql/pragma.h b/src/box/sql/pragma.h index 02895b0..dcacbd9 100644 --- a/src/box/sql/pragma.h +++ b/src/box/sql/pragma.h @@ -6,19 +6,14 @@ /* The various pragma types */ #define PragTyp_COLLATION_LIST 3 -#define PragTyp_FLAG 5 #define PragTyp_FOREIGN_KEY_LIST 9 #define PragTyp_INDEX_INFO 10 #define PragTyp_INDEX_LIST 11 #define PragTyp_STATS 15 #define PragTyp_TABLE_INFO 17 -#define PragTyp_DEFAULT_ENGINE 25 -#define PragTyp_COMPOUND_SELECT_LIMIT 26 /* Property flags associated with various pragma. */ #define PragFlg_NeedSchema 0x01 /* Force schema load before running */ -#define PragFlg_NoColumns 0x02 /* OP_ResultRow called with zero columns */ -#define PragFlg_NoColumns1 0x04 /* zero columns if RHS argument is present */ #define PragFlg_Result0 0x10 /* Acts as query when no argument */ #define PragFlg_Result1 0x20 /* Acts as query when has one argument */ #define PragFlg_SchemaOpt 0x40 /* Schema restricts name search if present */ @@ -93,57 +88,6 @@ static const char *const pragCName[] = { /* 55 */ "TEXT", /* 56 */ "match", /* 57 */ "TEXT", - /* Used by: count_changes */ - /* 58 */ "count_changes", - /* 59 */ "INTEGER", - /* Used by: defer_foreign_keys */ - /* 60 */ "defer_foreign_keys", - /* 61 */ "INTEGER", - /* Used by: full_column_names */ - /* 62 */ "full_column_names", - /* 63 */ "INTEGER", - /* Used by: parser_trace */ - /* 64 */ "parser_trace", - /* 65 */ "INTEGER", - /* Used by: recursive_triggers */ - /* 66 */ "recursive_triggers", - /* 67 */ "INTEGER", - /* Used by: reverse_unordered_selects */ - /* 68 */ "reverse_unordered_selects", - /* 69 */ "INTEGER", - /* Used by: select_trace */ - /* 70 */ "select_trace", - /* 71 */ "INTEGER", - /* Used by: short_column_names */ - /* 72 */ "short_column_names", - /* 73 */ "INTEGER", - /* Used by: sql_compound_select_limit */ - /* 74 */ "sql_compound_select_limit", - /* 75 */ "INTEGER", - /* Used by: sql_default_engine */ - /* 76 */ "sql_default_engine", - /* 77 */ "TEXT", - /* Used by: sql_trace */ - /* 78 */ "sql_trace", - /* 79 */ "INTEGER", - /* Used by: vdbe_addoptrace */ - /* 80 */ "vdbe_addoptrace", - /* 81 */ "INTEGER", - /* Used by: vdbe_debug */ - /* 82 */ "vdbe_debug", - /* 83 */ "INTEGER", - /* Used by: vdbe_eqp */ - /* 84 */ "vdbe_eqp", - /* 85 */ "INTEGER", - /* Used by: vdbe_listing */ - /* 86 */ "vdbe_listing", - /* 87 */ "INTEGER", - /* Used by: vdbe_trace */ - /* 88 */ "vdbe_trace", - /* 89 */ "INTEGER", - /* Used by: where_trace */ - /* 90 */ "where_trace", - /* 91 */ "INTEGER", }; /* Definitions of all built-in pragmas */ @@ -165,27 +109,12 @@ static const PragmaName aPragmaName[] = { /* ePragFlg: */ PragFlg_Result0, /* ColNames: */ 38, 2, /* iArg: */ 0}, - { /* zName: */ "count_changes", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 60, 1, - /* iArg: */ SQL_CountRows}, - { /* zName: */ "defer_foreign_keys", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 62, 1, - /* iArg: */ SQL_DeferFKs}, { /* zName: */ "foreign_key_list", /* ePragTyp: */ PragTyp_FOREIGN_KEY_LIST, /* ePragFlg: */ PragFlg_NeedSchema | PragFlg_Result1 | PragFlg_SchemaOpt, /* ColNames: */ 42, 8, /* iArg: */ 0}, - { /* zName: */ "full_column_names", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 64, 1, - /* iArg: */ SQL_FullColNames}, { /* zName: */ "index_info", /* ePragTyp: */ PragTyp_INDEX_INFO, /* ePragFlg: */ @@ -198,52 +127,6 @@ static const PragmaName aPragmaName[] = { PragFlg_NeedSchema | PragFlg_Result1 | PragFlg_SchemaOpt, /* ColNames: */ 32, 3, /* iArg: */ 0}, -#if defined(SQL_DEBUG) - { /* zName: */ "parser_trace", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 64, 1, - /* iArg: */ PARSER_TRACE_FLAG}, -#endif - { /* zName: */ "recursive_triggers", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 66, 1, - /* iArg: */ SQL_RecTriggers}, - { /* zName: */ "reverse_unordered_selects", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 68, 1, - /* iArg: */ SQL_ReverseOrder}, -#if defined(SQL_DEBUG) - { /* zName: */ "select_trace", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 70, 1, - /* iArg: */ SQL_SelectTrace}, -#endif - { /* zName: */ "short_column_names", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 72, 1, - /* iArg: */ SQL_ShortColNames}, - { /* zName: */ "sql_compound_select_limit", - /* ePragTyp: */ PragTyp_COMPOUND_SELECT_LIMIT, - /* ePragFlg: */ PragFlg_Result0, - /* ColNames: */ 74, 1, - /* iArg: */ 0}, - { /* zName: */ "sql_default_engine", - /* ePragTyp: */ PragTyp_DEFAULT_ENGINE, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 76, 1, - /* iArg: */ 0}, -#if defined(SQL_DEBUG) - { /* zName: */ "sql_trace", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 78, 1, - /* iArg: */ SQL_SqlTrace}, -#endif { /* zName: */ "stats", /* ePragTyp: */ PragTyp_STATS, /* ePragFlg: */ @@ -256,38 +139,5 @@ static const PragmaName aPragmaName[] = { PragFlg_NeedSchema | PragFlg_Result1 | PragFlg_SchemaOpt, /* ColNames: */ 0, 6, /* iArg: */ 0}, -#if defined(SQL_DEBUG) - { /* zName: */ "vdbe_addoptrace", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 80, 1, - /* iArg: */ SQL_VdbeAddopTrace}, - { /* zName: */ "vdbe_debug", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 82, 1, - /* iArg: */ - SQL_SqlTrace | SQL_VdbeListing | SQL_VdbeTrace}, - { /* zName: */ "vdbe_eqp", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 84, 1, - /* iArg: */ SQL_VdbeEQP}, - { /* zName: */ "vdbe_listing", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 86, 1, - /* iArg: */ SQL_VdbeListing}, - { /* zName: */ "vdbe_trace", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 88, 1, - /* iArg: */ SQL_VdbeTrace}, - { /* zName: */ "where_trace", - /* ePragTyp: */ PragTyp_FLAG, - /* ePragFlg: */ PragFlg_Result0 | PragFlg_NoColumns1, - /* ColNames: */ 90, 1, - /* iArg: */ SQL_WhereTrace}, -#endif }; /* Number of pragmas: 36 on by default, 47 total. */ diff --git a/src/box/sql/select.c b/src/box/sql/select.c index 8f93edd..b81c7af 100644 --- a/src/box/sql/select.c +++ b/src/box/sql/select.c @@ -1760,7 +1760,6 @@ generateColumnNames(Parse * pParse, /* Parser context */ Vdbe *v = pParse->pVdbe; int i, j; sql *db = pParse->db; - int fullNames, shortNames; /* If this is an EXPLAIN, skip this step */ if (pParse->explain) { return; @@ -1778,8 +1777,6 @@ generateColumnNames(Parse * pParse, /* Parser context */ } assert(pTabList != 0); pParse->colNamesSet = 1; - fullNames = (pParse->sql_flags & SQL_FullColNames) != 0; - shortNames = (pParse->sql_flags & SQL_ShortColNames) != 0; sqlVdbeSetNumCols(v, pEList->nExpr); uint32_t var_count = 0; for (i = 0; i < pEList->nExpr; i++) { @@ -1806,12 +1803,7 @@ generateColumnNames(Parse * pParse, /* Parser context */ struct space_def *space_def = pTabList->a[j].space->def; assert(iCol >= 0 && iCol < (int)space_def->field_count); zCol = space_def->fields[iCol].name; - if (!shortNames && !fullNames) { - sqlVdbeSetColName(v, i, COLNAME_NAME, - sqlDbStrDup(db, - pEList->a[i].zSpan), - SQL_DYNAMIC); - } else if (fullNames) { + if ((pParse->sql_flags & SQL_FullColNames) != 0) { char *zName = 0; zName = sqlMPrintf(db, "%s.%s", space_def->name, zCol); @@ -2022,8 +2014,7 @@ sqlResultSetOfSelect(Parse * pParse, Select * pSelect) sql *db = pParse->db; uint32_t saved_flags = pParse->sql_flags; - pParse->sql_flags |= ~SQL_FullColNames; - pParse->sql_flags &= SQL_ShortColNames; + pParse->sql_flags = 0; sqlSelectPrep(pParse, pSelect, 0); if (pParse->is_aborted) return NULL; @@ -4926,8 +4917,7 @@ selectExpander(Walker * pWalker, Select * p) struct ExprList_item *a = pEList->a; ExprList *pNew = 0; uint32_t flags = pParse->sql_flags; - int longNames = (flags & SQL_FullColNames) != 0 - && (flags & SQL_ShortColNames) == 0; + int longNames = (flags & SQL_FullColNames) != 0; for (k = 0; k < pEList->nExpr; k++) { pE = a[k].pExpr; diff --git a/src/box/sql/sqlInt.h b/src/box/sql/sqlInt.h index 9a2b5c8..3a412ad 100644 --- a/src/box/sql/sqlInt.h +++ b/src/box/sql/sqlInt.h @@ -1154,10 +1154,6 @@ struct sql { * Possible values for the sql.flags. */ -#define SQL_ShortColNames 0x00000040 /* Show short columns names */ -#define SQL_CountRows 0x00000080 /* Count rows changed by INSERT, */ - /* DELETE, or UPDATE and return */ - /* the count using a callback. */ #define SQL_AutoIndex 0x00100000 /* Enable automatic indexes */ #define SQL_EnableTrigger 0x01000000 /* True to enable triggers */ @@ -3928,8 +3924,6 @@ int sql_rem_int(int64_t lhs, bool is_lhs_neg, int64_t rhs, bool is_rhs_neg, int64_t *res, bool *is_res_neg); -u8 sqlGetBoolean(const char *z, u8); - const void *sqlValueText(sql_value *); int sqlValueBytes(sql_value *); void sqlValueSetStr(sql_value *, int, const void *, diff --git a/src/box/sql/update.c b/src/box/sql/update.c index 2d7ebf8..1e2bbe1 100644 --- a/src/box/sql/update.c +++ b/src/box/sql/update.c @@ -76,7 +76,6 @@ sqlUpdate(Parse * pParse, /* The parser context */ int aiCurOnePass[2]; /* The write cursors opened by WHERE_ONEPASS */ /* Register Allocations */ - int regRowCount = 0; /* A count of rows changed */ int regOldPk = 0; int regNewPk = 0; int regNew = 0; /* Content of the NEW.* table in triggers */ @@ -269,13 +268,6 @@ sqlUpdate(Parse * pParse, /* The parser context */ sqlWhereEnd(pWInfo); - /* Initialize the count of updated rows - */ - if ((pParse->sql_flags & SQL_CountRows) != 0 && - pParse->triggered_space == NULL) { - regRowCount = ++pParse->nMem; - sqlVdbeAddOp2(v, OP_Integer, 0, regRowCount); - } labelBreak = sqlVdbeMakeLabel(v); /* Top of the update loop */ if (okOnePass) { @@ -470,13 +462,6 @@ sqlUpdate(Parse * pParse, /* The parser context */ fk_constraint_emit_actions(pParse, space, regOldPk, aXRef); } - /* Increment the row counter - */ - if ((pParse->sql_flags & SQL_CountRows) != 0 && - pParse->triggered_space == NULL) { - sqlVdbeAddOp2(v, OP_AddImm, regRowCount, 1); - } - vdbe_code_row_trigger(pParse, trigger, TK_UPDATE, pChanges, TRIGGER_AFTER, space, regOldPk, on_error, labelContinue); @@ -493,17 +478,6 @@ sqlUpdate(Parse * pParse, /* The parser context */ } sqlVdbeResolveLabel(v, labelBreak); - /* Return the number of rows that were changed. */ - if ((pParse->sql_flags & SQL_CountRows) != 0 && - pParse->triggered_space == NULL) { - sqlVdbeAddOp2(v, OP_ResultRow, regRowCount, 1); - sqlVdbeSetNumCols(v, 1); - sqlVdbeSetColName(v, 0, COLNAME_NAME, "rows updated", - SQL_STATIC); - sqlVdbeSetColName(v, 0, COLNAME_DECLTYPE, "INTEGER", - SQL_STATIC); - } - update_cleanup: sqlSrcListDelete(db, pTabList); sql_expr_list_delete(db, pChanges); diff --git a/src/box/sql/vdbe.c b/src/box/sql/vdbe.c index f501982..d2221cb 100644 --- a/src/box/sql/vdbe.c +++ b/src/box/sql/vdbe.c @@ -1412,31 +1412,7 @@ case OP_ResultRow: { assert(pOp->p1>0); assert(pOp->p1+pOp->p2<=(p->nMem+1 - p->nCursor)+1); - /* If this statement has violated immediate foreign key constraints, do - * not return the number of rows modified. And do not RELEASE the statement - * transaction. It needs to be rolled back. - */ - if (sqlVdbeCheckFk(p, 0) != 0) { - assert((p->sql_flags & SQL_CountRows) != 0); - goto abort_due_to_error; - } - - /* If the SQL_CountRows flag is set in sql.flags mask, then - * DML statements invoke this opcode to return the number of rows - * modified to the user. This is the only way that a VM that - * opens a statement transaction may invoke this opcode. - * - * In case this is such a statement, close any statement transaction - * opened by this VM before returning control to the user. This is to - * ensure that statement-transactions are always nested, not overlapping. - * If the open statement-transaction is not closed here, then the user - * may step another VM that opens its own statement transaction. This - * may lead to overlapping statement transactions. - * - * The statement transaction is never a top-level transaction. Hence - * the RELEASE call below can never fail. - */ - assert(p->iStatement == 0 || (p->sql_flags & SQL_CountRows) != 0); + assert(p->iStatement == 0); rc = sqlVdbeCloseStatement(p, SAVEPOINT_RELEASE); assert(rc==0); diff --git a/test/sql-tap/analyze4.test.lua b/test/sql-tap/analyze4.test.lua index 7d7498f..cc530ce 100755 --- a/test/sql-tap/analyze4.test.lua +++ b/test/sql-tap/analyze4.test.lua @@ -70,9 +70,9 @@ test:do_test( function() return test:execsql([[ UPDATE t1 SET b='x' WHERE a%2; --- pragma vdbe_debug=1; +-- set vdbe_debug=true; ANALYZE; --- pragma vdbe_debug=0; +-- set vdbe_debug=false; SELECT "idx", "stat" FROM "_sql_stat1" WHERE "tbl"='T1' ORDER BY "idx"; ]]) end, { diff --git a/test/sql-tap/autoinc.test.lua b/test/sql-tap/autoinc.test.lua index 39e4796..a0eef92 100755 --- a/test/sql-tap/autoinc.test.lua +++ b/test/sql-tap/autoinc.test.lua @@ -640,7 +640,7 @@ test:do_test( -- </autoinc-10.3> }) -test:catchsql(" pragma recursive_triggers = off ") +test:catchsql("SET recursive_triggers = false") -- Ticket #3928. Make sure that triggers to not make extra slots in -- the sql_SEQUENCE table. -- diff --git a/test/sql-tap/colname.test.lua b/test/sql-tap/colname.test.lua index 253497c..a14230b 100755 --- a/test/sql-tap/colname.test.lua +++ b/test/sql-tap/colname.test.lua @@ -1,6 +1,6 @@ #!/usr/bin/env tarantool test = require("sqltester") -test:plan(63) +test:plan(62) --!./tcltestrunner.lua -- 2008 July 15 @@ -47,24 +47,14 @@ local function lreplace(arr, pos, len, val) end test:do_test( - "colname-1.1", - function() - return test:execsql "PRAGMA short_column_names" - end, { - -- <colname-1.1> - 1 - -- </colname-1.1> - }) - -test:do_test( "colname-1.2", function() - return test:execsql "PRAGMA full_column_names" - end, { + return box.space._vsql_settings:get("full_column_names")[2] + end, -- <colname-1.2> - 0 + false -- </colname-1.2> - }) + ) -- Tests for then short=ON and full=any -- @@ -177,8 +167,7 @@ test:do_test( "colname-3.1", function() test:execsql [[ - PRAGMA short_column_names='OFF'; - PRAGMA full_column_names='OFF'; + SET full_column_names = false; CREATE VIEW v3 AS SELECT tabC.a, txyZ.x, * FROM tabc, txyz ORDER BY 1 LIMIT 1; CREATE VIEW v4 AS SELECT tabC.a, txyZ.x, tboTh.a, tbotH.x, * @@ -199,7 +188,7 @@ test:do_execsql2_test( SELECT Tabc.a, tAbc.b, taBc.c FROM tabc ]], { -- <colname-3.2> - "Tabc.a", 1, "tAbc.b", 2, "taBc.c", 3 + "A", 1, "B", 2, "C", 3 -- </colname-3.2> }) @@ -209,7 +198,7 @@ test:do_execsql2_test( SELECT +tabc.a, -tabc.b, tabc.c FROM tabc ]], { -- <colname-3.3> - "+tabc.a", 1, "-tabc.b", -2, "tabc.c", 3 + "+tabc.a", 1, "-tabc.b", -2, "C", 3 -- </colname-3.3> }) @@ -229,7 +218,7 @@ test:do_execsql2_test( SELECT Tabc.a, Txyz.x, * FROM tabc, txyz; ]], { -- <colname-3.5> - "Tabc.a", 1, "Txyz.x", 4, "A", 1, "B", 2, "C", 3, "X", 4, "Y", 5, "Z", 6 + "A", 1, "X", 4, "A", 1, "B", 2, "C", 3, "X", 4, "Y", 5, "Z", 6 -- </colname-3.5> }) @@ -259,7 +248,7 @@ test:do_execsql2_test( SELECT v1.a, * FROM v1 ORDER BY 2; ]], { -- <colname-3.8> - "v1.a",1,"A",1,"X",4,"A_1",1,"B",2,"C",3,"X_1",4,"Y",5,"Z",6 + "A",1,"A",1,"X",4,"A_1",1,"B",2,"C",3,"X_1",4,"Y",5,"Z",6 -- </colname-3.8> }) @@ -298,8 +287,7 @@ test:do_test( "colname-4.1", function() test:execsql [[ - PRAGMA short_column_names='OFF'; - PRAGMA full_column_names='ON'; + SET full_column_names = true; CREATE VIEW v5 AS SELECT tabC.a, txyZ.x, * FROM tabc, txyz ORDER BY 1 LIMIT 1; CREATE VIEW v6 AS SELECT tabC.a, txyZ.x, tboTh.a, tbotH.x, * @@ -456,8 +444,7 @@ test:do_test( -- instead of reconnect to database -- we are just turning settings to default state test:execsql([[ - PRAGMA short_column_names='ON'; - PRAGMA full_column_names='OFF'; + SET full_column_names = false; ]]) test:execsql [=[ CREATE TABLE t6(a INT primary key, "'a'" INT, """a""" INT, "[a]" INT, "`a`" INT); diff --git a/test/sql-tap/fkey2.test.lua b/test/sql-tap/fkey2.test.lua index 264616d..9963f93 100755 --- a/test/sql-tap/fkey2.test.lua +++ b/test/sql-tap/fkey2.test.lua @@ -450,7 +450,7 @@ test:do_execsql_test( test:do_execsql_test( "fkey2-4.2", [[ - PRAGMA recursive_triggers = off; + SET recursive_triggers = false; DELETE FROM t1 WHERE node = 1; SELECT node FROM t1; ]], { @@ -473,7 +473,7 @@ test:do_execsql_test( test:do_execsql_test( "fkey2-4.4", [[ - PRAGMA recursive_triggers = on; + SET recursive_triggers = true; DROP TABLE t2; DROP TABLE t1; CREATE TABLE t1( diff --git a/test/sql-tap/gh2548-select-compound-limit.test.lua b/test/sql-tap/gh2548-select-compound-limit.test.lua index f578870..7f957b5 100755 --- a/test/sql-tap/gh2548-select-compound-limit.test.lua +++ b/test/sql-tap/gh2548-select-compound-limit.test.lua @@ -64,7 +64,7 @@ test:do_catchsql_test( test:do_execsql_test( "gh2548-select-compound-limit-3.1", [[ - pragma sql_compound_select_limit + SELECT "value" FROM "_vsql_settings" WHERE "name" = 'sql_compound_select_limit' ]], { -- <gh2548-select-compound-limit-3.1> 30 @@ -73,7 +73,8 @@ test:do_execsql_test( test:do_execsql_test( "gh2548-select-compound-limit-3.2", [[ - pragma sql_compound_select_limit=31 + SET sql_compound_select_limit = 31; + SELECT "value" FROM "_vsql_settings" WHERE "name" = 'sql_compound_select_limit' ]], { -- <gh2548-select-compound-limit-3.2> 31 @@ -90,7 +91,8 @@ test:do_execsql_test( test:do_execsql_test( "gh2548-select-compound-limit-3.4", [[ - pragma sql_compound_select_limit=0 + SET sql_compound_select_limit = 0; + SELECT "value" FROM "_vsql_settings" WHERE "name" = 'sql_compound_select_limit'; ]], { -- <gh2548-select-compound-limit-3.4> 0 diff --git a/test/sql-tap/lua/sqltester.lua b/test/sql-tap/lua/sqltester.lua index 0f34114..c50e3fd 100644 --- a/test/sql-tap/lua/sqltester.lua +++ b/test/sql-tap/lua/sqltester.lua @@ -413,7 +413,7 @@ box.cfg{ } local engine = test_run and test_run:get_cfg('engine') or 'memtx' -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') function test.engine(self) return engine diff --git a/test/sql-tap/misc1.test.lua b/test/sql-tap/misc1.test.lua index b84093e..33b678b 100755 --- a/test/sql-tap/misc1.test.lua +++ b/test/sql-tap/misc1.test.lua @@ -219,7 +219,7 @@ test:do_test( local r = test:execsql([[ CREATE TABLE t1(a TEXT primary KEY); INSERT INTO t1 VALUES('hi'); - PRAGMA full_column_names=on; + SET full_column_names = true; --SELECT rowid, * FROM t1; SELECT * FROM t1; ]]) diff --git a/test/sql-tap/pragma.test.lua b/test/sql-tap/pragma.test.lua index b3821dc..d6f8b54 100755 --- a/test/sql-tap/pragma.test.lua +++ b/test/sql-tap/pragma.test.lua @@ -1,7 +1,7 @@ #!/usr/bin/env tarantool test = require("sqltester") -test:plan(23) +test:plan(10) test:do_catchsql_test( "pragma-1.3", @@ -19,66 +19,7 @@ test:do_catchsql_test( [[ pragma sql_default_engine='creepy'; ]], { - 1, "Space engine 'creepy' does not exist" -}) - -test:do_catchsql_test( - "pragma-2.2", - [[ - pragma sql_default_engine='vinyl'; - ]], { - 0 -}) - -test:do_catchsql_test( - "pragma-2.3", - [[ - pragma sql_default_engine='memtx'; - ]], { - 0 -}) - -test:do_catchsql_test( - "pragma-2.4", - [[ - pragma sql_default_engine 'memtx'; - ]], { - 1, "Syntax error near ''memtx''" -}) - -test:do_catchsql_test( - "pragma-2.5", - [[ - pragma sql_default_engine 1; - ]], { - 1, "Syntax error near '1'" -}) - --- --- gh-3832: Some statements do not return column type --- --- Check that "PRAGMA sql_default_engine" called without arguments --- returns currently set sql_default_engine. -test:do_execsql_test( - "pragma-3.1", - [[ - pragma sql_default_engine='vinyl'; - pragma sql_default_engine; - ]], { - -- <pragma-3.1> - 'vinyl' - -- </pragma-3.1> -}) - -test:do_execsql_test( - "pragma-3.2", - [[ - pragma sql_default_engine='memtx'; - pragma sql_default_engine; - ]], { - -- <pragma-3.2> - 'memtx' - -- </pragma-3.2> + 1, "Pragma 'SQL_DEFAULT_ENGINE' does not exist" }) -- @@ -187,76 +128,4 @@ test:do_execsql_test( -- </pragma-8.4> }) ---- ---- pragma sql_default_engine accepts string values and rejects IDs ---- -test:do_catchsql_test( - "pragma-9.1", - [[ - pragma sql_default_engine(the_engine); - ]], { - -- <pragma-9.1> - 1, "Illegal parameters, string value is expected" - -- </pragma-9.1> -}) - -test:do_catchsql_test( - "pragma-9.2", - [[ - pragma sql_default_engine(THE_ENGINE); - ]], { - -- <pragma-9.2> - 1, "Illegal parameters, string value is expected" - -- </pragma-9.2> -}) - -test:do_catchsql_test( - "pragma-9.3", - [[ - pragma sql_default_engine("THE_ENGINE"); - ]], { - -- <pragma-9.3> - 1, "Illegal parameters, string value is expected" - -- </pragma-9.3> -}) - -test:do_catchsql_test( - "pragma-9.4", - [[ - pragma sql_default_engine('THE_ENGINE'); - ]], { - -- <pragma-9.4> - 1, "Space engine 'THE_ENGINE' does not exist" - -- </pragma-9.4> -}) - -test:do_catchsql_test( - "pragma-9.5", - [[ - pragma sql_default_engine(memtx); - ]], { - -- <pragma-9.5> - 1, "Illegal parameters, string value is expected" - -- </pragma-9.5> -}) - -test:do_catchsql_test( - "pragma-9.6", - [[ - pragma sql_default_engine("memtx"); - ]], { - -- <pragma-9.6> - 1, "Illegal parameters, string value is expected" - -- </pragma-9.6> -}) - -test:do_execsql_test( - "pragma-9.7", - [[ - pragma sql_default_engine('memtx'); - ]], { - -- <pragma-9.7> - -- </pragma-9.7> -}) - test:finish_test() diff --git a/test/sql-tap/select1.test.lua b/test/sql-tap/select1.test.lua index 4bbfbd6..2b972bd 100755 --- a/test/sql-tap/select1.test.lua +++ b/test/sql-tap/select1.test.lua @@ -916,7 +916,7 @@ test:do_catchsql2_test( test:do_test( "select1-6.1.1", function() - test:execsql "PRAGMA full_column_names=on" + test:execsql "SET full_column_names = true" return test:catchsql2 "SELECT f1 FROM test1 ORDER BY f2" end, { -- <select1-6.1.1> @@ -940,7 +940,7 @@ test:do_catchsql2_test( SELECT * FROM test1 WHERE f1==11 ]], { -- <select1-6.1.3> - 0, {"F1", 11, "F2", 22} + 0, {"TEST1.F1", 11, "TEST1.F2", 22} -- </select1-6.1.3> }) @@ -952,11 +952,11 @@ test:do_test( msg = test:execsql2 "SELECT DISTINCT * FROM test1 WHERE f1==11" end) v = v == true and {0} or {1} - test:execsql "PRAGMA full_column_names=off" + test:execsql "SET full_column_names = false" return table.insert(v,msg) or v end, { -- <select1-6.1.4> - 0, {"F1", 11, "F2", 22} + 0, {"TEST1.F1", 11, "TEST1.F2", 22} -- </select1-6.1.4> }) @@ -1043,13 +1043,13 @@ test:do_catchsql2_test( test:do_test( "select1-6.5.1", function() - test:execsql2 "PRAGMA full_column_names=on" + test:execsql2 "SET full_column_names = true" local msg v = pcall( function () msg = test:execsql2 "SELECT test1.f1+F2 FROM test1 ORDER BY f2" end) v = v == true and {0} or {1} - test:execsql2 "PRAGMA full_column_names=off" + test:execsql2 "SET full_column_names = false" return table.insert(v,msg) or v end, { -- <select1-6.5.1> @@ -1124,15 +1124,14 @@ test:do_test( "select1-6.9.3", function() test:execsql [[ - PRAGMA short_column_names='OFF'; - PRAGMA full_column_names='OFF'; + SET full_column_names = false; ]] return test:execsql2 [[ SELECT test1 . f1, test1 . f2 FROM test1 LIMIT 1 ]] end, { -- <select1-6.9.3> - "test1 . f1", 11, "test1 . f2", 22 + "F1", 11, "F2", 22 -- </select1-6.9.3> }) @@ -1140,8 +1139,7 @@ test:do_test( "select1-6.9.4", function() test:execsql [[ - PRAGMA short_column_names='OFF'; - PRAGMA full_column_names='ON'; + SET full_column_names = true; ]] return test:execsql2 [[ SELECT test1 . f1, test1 . f2 FROM test1 LIMIT 1 @@ -1156,8 +1154,7 @@ test:do_test( "select1-6.9.5", function() test:execsql [[ - PRAGMA short_column_names='OFF'; - PRAGMA full_column_names='ON'; + SET full_column_names = true; ]] return test:execsql2 [[ SELECT 123.45; @@ -1238,8 +1235,7 @@ test:do_test( "select1-6.9.11", function() test:execsql [[ - PRAGMA short_column_names='ON'; - PRAGMA full_column_names='ON'; + SET full_column_names = true; ]] return test:execsql2 [[ SELECT a.f1, b.f2 FROM test1 a, test1 b LIMIT 1 @@ -1264,8 +1260,7 @@ test:do_test( "select1-6.9.13", function() test:execsql [[ - PRAGMA short_column_names='ON'; - PRAGMA full_column_names='OFF'; + SET full_column_names = false; ]] return test:execsql2 [[ SELECT a.f1, b.f1 FROM test1 a, test1 b LIMIT 1 @@ -1290,8 +1285,7 @@ test:do_test( "select1-6.9.15", function() test:execsql [[ - PRAGMA short_column_names='OFF'; - PRAGMA full_column_names='ON'; + SET full_column_names = true; ]] return test:execsql2 [[ SELECT a.f1, b.f1 FROM test1 a, test1 b LIMIT 1 @@ -1313,8 +1307,7 @@ test:do_execsql2_test( }) test:execsql [[ - PRAGMA short_column_names='ON'; - PRAGMA full_column_names='OFF'; + SET full_column_names = false; ]] test:do_catchsql2_test( "select1-6.10", diff --git a/test/sql-tap/tkt3731.test.lua b/test/sql-tap/tkt3731.test.lua index 454cf67..0d96372 100755 --- a/test/sql-tap/tkt3731.test.lua +++ b/test/sql-tap/tkt3731.test.lua @@ -22,7 +22,7 @@ test:plan(3) -- The tests in this file were written before sql supported recursive -- trigger invocation, and some tests depend on that to pass. So disable -- recursive triggers for this file. -test:catchsql " pragma recursive_triggers = off " +test:catchsql("SET recursive_triggers = false") test:do_execsql_test( "tkt3731-1.1", [[ diff --git a/test/sql-tap/trigger2.test.lua b/test/sql-tap/trigger2.test.lua index 6e60050..a96482b 100755 --- a/test/sql-tap/trigger2.test.lua +++ b/test/sql-tap/trigger2.test.lua @@ -58,7 +58,7 @@ test:plan(26) -- The tests in this file were written before sql supported recursive -- trigger invocation, and some tests depend on that to pass. So disable -- recursive triggers for this file. -test:catchsql " pragma recursive_triggers = off " +test:catchsql("SET recursive_triggers = false") -- 1. ii = 0 tbl_definitions = { "CREATE TABLE tbl (id INT PRIMARY KEY AUTOINCREMENT, a INTEGER UNIQUE, b INT );", diff --git a/test/sql-tap/triggerC.test.lua b/test/sql-tap/triggerC.test.lua index cf7dd7b..e93fde7 100755 --- a/test/sql-tap/triggerC.test.lua +++ b/test/sql-tap/triggerC.test.lua @@ -1,6 +1,6 @@ #!/usr/bin/env tarantool test = require("sqltester") -test:plan(48) +test:plan(45) --!./tcltestrunner.lua -- 2009 August 24 @@ -36,12 +36,9 @@ testprefix = "triggerC" -- REPLACE conflict resolution. And that they are not fired -- if recursive triggers are not enabled. -- --- triggerC-6.*: Test that the recursive_triggers pragma returns correct --- results when invoked without an argument. --- -- Enable recursive triggers for this file. -- -test:execsql " PRAGMA recursive_triggers = on " +test:execsql("SET recursive_triggers = true;") --------------------------------------------------------------------------- -- This block of tests, triggerC-1.*, are not aimed at any specific -- property of the triggers sub-system. They were created to debug @@ -504,7 +501,7 @@ test:do_execsql_test( test:do_execsql_test( "triggerC-5.3.0", [[ - PRAGMA recursive_triggers = off + SET recursive_triggers = false ]], { -- <triggerC-5.3.0> @@ -534,49 +531,13 @@ test:do_execsql_test( test:do_execsql_test( "triggerC-5.3.8", [[ - PRAGMA recursive_triggers = on + SET recursive_triggers = true ]], { -- <triggerC-5.3.8> -- </triggerC-5.3.8> }) ---------------------------------------------------------------------------- --- This block of tests, triggerC-6.*, tests that "PRAGMA recursive_triggers" --- statements return the current value of the recursive triggers flag. --- -test:do_execsql_test( - "triggerC-6.1", - [[ - PRAGMA recursive_triggers - ]], { - -- <triggerC-6.1> - 1 - -- </triggerC-6.1> - }) - -test:do_execsql_test( - "triggerC-6.2", - [[ - PRAGMA recursive_triggers = off; - PRAGMA recursive_triggers; - ]], { - -- <triggerC-6.2> - 0 - -- </triggerC-6.2> - }) - -test:do_execsql_test( - "triggerC-6.3", - [[ - PRAGMA recursive_triggers = on; - PRAGMA recursive_triggers; - ]], { - -- <triggerC-6.3> - 1 - -- </triggerC-6.3> - }) - -- MUST_WORK_TEST -- #------------------------------------------------------------------------- -- # Test some of the "undefined behaviour" associated with triggers. The @@ -890,7 +851,7 @@ test:execsql( test:do_execsql_test( "triggerC-13.1", [[ - PRAGMA recursive_triggers = 'ON'; + SET recursive_triggers = true; CREATE TABLE t12(id INTEGER PRIMARY KEY, a INT, b INT); INSERT INTO t12 VALUES(1, 1, 2); CREATE TRIGGER tr12 AFTER UPDATE ON t12 FOR EACH ROW BEGIN @@ -974,7 +935,7 @@ test:do_execsql_test( test:do_execsql_test( "triggerC-15.1.1", [[ - PRAGMA recursive_triggers = 1; + SET recursive_triggers = true; CREATE TABLE node( id int not null primary key, pid int not null default 0, diff --git a/test/sql-tap/update.test.lua b/test/sql-tap/update.test.lua index 07e4e43..9efc0cf 100755 --- a/test/sql-tap/update.test.lua +++ b/test/sql-tap/update.test.lua @@ -124,13 +124,12 @@ test:do_execsql_test("update-3.6", [[ }) test:do_test("update-3.7", function() - test:execsql "PRAGMA count_changes=on" - return test:execsql "UPDATE test1 SET f2=f2/3 WHERE f1<=5" -end, { + return box.execute("UPDATE test1 SET f2=f2/3 WHERE f1<=5").row_count +end, -- <update-3.7> 5 -- </update-3.7> -}) +) test:do_execsql_test("update-3.8", [[ SELECT f1,f2 FROM test1 ORDER BY f1 @@ -144,7 +143,6 @@ test:do_execsql_test("update-3.9", [[ UPDATE test1 SET f2=f2/3 WHERE f1>5 ]], { -- <update-3.9> - 5 -- </update-3.9> }) @@ -162,7 +160,6 @@ test:do_execsql_test("update-3.11", [[ UPDATE test1 SET F2=f1, F1=f2 ]], { -- <update-3.11> - 10 -- </update-3.11> }) @@ -175,7 +172,6 @@ test:do_execsql_test("update-3.12", [[ }) test:do_test("update-3.13", function() - test:execsql "PRAGMA count_changes=off" return test:execsql "UPDATE test1 SET F2=f1, F1=f2" end, { -- <update-3.13> @@ -255,16 +251,13 @@ end, { }) test:do_execsql_test("update-4.6", [[ - PRAGMA count_changes=on; UPDATE test1 SET f1=f1-1 WHERE f1<=100 and f2==128; ]], { -- <update-4.6> - 2 -- </update-4.6> }) test:do_execsql_test("update-4.7", [[ - PRAGMA count_changes=off; SELECT f1,f2 FROM test1 ORDER BY f1,f2 ]], { -- <update-4.7> @@ -392,16 +385,13 @@ test:do_execsql_test("update-5.5.5", [[ }) test:do_execsql_test("update-5.6", [[ - PRAGMA count_changes=on; UPDATE test1 SET f1=f1-1 WHERE f1<=100 and f2==128; ]], { -- <update-5.6> - 2 -- </update-5.6> }) test:do_execsql_test("update-5.6.1", [[ - PRAGMA count_changes=off; SELECT f1,f2 FROM test1 ORDER BY f1,f2 ]], { -- <update-5.6.1> @@ -443,7 +433,6 @@ test:do_execsql_test("update-5.6.5", [[ -- Repeat the previous sequence of tests with a different index. -- ---test:execsql "PRAGMA synchronous='FULL'" test:do_test("update-6.0", function() test:execsql "DROP INDEX idx1 ON test1" test:execsql "CREATE INDEX idx1 ON test1(f2)" diff --git a/test/sql-tap/whereA.test.lua b/test/sql-tap/whereA.test.lua index b82575f..0341b65 100755 --- a/test/sql-tap/whereA.test.lua +++ b/test/sql-tap/whereA.test.lua @@ -14,7 +14,7 @@ test:plan(17) -- ------------------------------------------------------------------------- -- This file implements regression tests for sql library. The --- focus of this file is testing the reverse_select_order pragma. +-- focus of this file is testing the reverse_select_order option. -- -- $Id: whereA.test,v 1.3 2009/06/10 19:33:29 drh Exp $ -- ["set","testdir",[["file","dirname",["argv0"]]]] @@ -39,7 +39,7 @@ test:do_test( "whereA-1.2", function() return test:execsql [[ - PRAGMA reverse_unordered_selects=1; + SET reverse_unordered_selects = true; SELECT * FROM t1; ]] end, { @@ -55,7 +55,7 @@ test:do_test( --db close --sql db test.db return test:execsql [[ - PRAGMA reverse_unordered_selects=1; + SET reverse_unordered_selects = true; SELECT * FROM t1; ]] end, { @@ -68,7 +68,7 @@ test:do_test( -- db close -- sql db test.db -- db eval { --- PRAGMA reverse_unordered_selects=1; +-- SET reverse_unordered_selects = true; -- SELECT * FROM t1 ORDER BY rowid; -- } -- } {1 2 3 2 hello world 3 4.53 {}} @@ -76,11 +76,11 @@ test:do_test( "whereA-1.6", function() return test:execsql [[ - PRAGMA reverse_unordered_selects; + SELECT "value" FROM "_vsql_settings" WHERE "name" = 'reverse_unordered_selects'; ]] end, { -- <whereA-1.6> - 1 + true -- </whereA-1.6> }) @@ -108,7 +108,7 @@ test:do_test( "whereA-2.1", function() return test:execsql [[ - PRAGMA reverse_unordered_selects=0; + SET reverse_unordered_selects = false; SELECT * FROM t1 WHERE a>0; ]] end, { @@ -121,7 +121,7 @@ test:do_test( "whereA-2.2", function() return test:execsql [[ - PRAGMA reverse_unordered_selects=1; + SET reverse_unordered_selects = true; SELECT * FROM t1 WHERE a>0; ]] end, { @@ -132,7 +132,7 @@ test:do_test( -- do_test whereA-2.3 { -- db eval { --- PRAGMA reverse_unordered_selects=1; +-- SET reverse_unordered_selects = true; -- SELECT * FROM t1 WHERE a>0 ORDER BY rowid; -- } -- } {1 2 3 2 hello world 3 4.53 {}} @@ -140,7 +140,7 @@ test:do_test( "whe:reA-3.1", function() return test:execsql [[ - PRAGMA reverse_unordered_selects=0; + SET reverse_unordered_selects = false; SELECT * FROM t1 WHERE b>0; ]] end, { @@ -153,7 +153,7 @@ test:do_test( "whereA-3.2", function() return test:execsql [[ - PRAGMA reverse_unordered_selects=1; + SET reverse_unordered_selects = true; SELECT * FROM t1 WHERE b>0; ]] end, { @@ -166,7 +166,7 @@ test:do_test( "whereA-3.3", function() return test:execsql [[ - PRAGMA reverse_unordered_selects=1; + SET reverse_unordered_selects = true; SELECT * FROM t1 WHERE b>0 ORDER BY b; ]] end, { diff --git a/test/sql/check-clear-ephemeral.result b/test/sql/check-clear-ephemeral.result index 7d0be5f..3b12457 100644 --- a/test/sql/check-clear-ephemeral.result +++ b/test/sql/check-clear-ephemeral.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -15,7 +15,7 @@ box.execute("CREATE TABLE t1(a INT,b INT,c INT,PRIMARY KEY(b,c));") - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("WITH RECURSIVE cnt(x) AS (VALUES(1) UNION ALL SELECT x+1 FROM cnt WHERE x<1000) INSERT INTO t1 SELECT x, x%40, x/40 FROM cnt;") --- diff --git a/test/sql/check-clear-ephemeral.test.lua b/test/sql/check-clear-ephemeral.test.lua index 929a6c9..2ecf11c 100644 --- a/test/sql/check-clear-ephemeral.test.lua +++ b/test/sql/check-clear-ephemeral.test.lua @@ -1,13 +1,13 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() -- create space box.execute("CREATE TABLE t1(a INT,b INT,c INT,PRIMARY KEY(b,c));") -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("WITH RECURSIVE cnt(x) AS (VALUES(1) UNION ALL SELECT x+1 FROM cnt WHERE x<1000) INSERT INTO t1 SELECT x, x%40, x/40 FROM cnt;") diff --git a/test/sql/checks.result b/test/sql/checks.result index 50347bc..a0f2e9e 100644 --- a/test/sql/checks.result +++ b/test/sql/checks.result @@ -11,7 +11,7 @@ test_run:cmd("push filter ".."'\\.lua.*:[0-9]+: ' to '.lua...\"]:<line>: '") engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/checks.test.lua b/test/sql/checks.test.lua index cde213f..eae4278 100644 --- a/test/sql/checks.test.lua +++ b/test/sql/checks.test.lua @@ -2,7 +2,7 @@ env = require('test_run') test_run = env.new() test_run:cmd("push filter ".."'\\.lua.*:[0-9]+: ' to '.lua...\"]:<line>: '") engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- -- gh-3272: Move SQL CHECK into server diff --git a/test/sql/clear.result b/test/sql/clear.result index afa6520..baeb15e 100644 --- a/test/sql/clear.result +++ b/test/sql/clear.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -19,7 +19,7 @@ box.execute("CREATE UNIQUE INDEX zoobar2 ON zoobar(c1, c4)") - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON;") +-- box.execute("SET vdbe_debug=ON;") -- Seed entry for i=1, 100 do box.execute(string.format("INSERT INTO zoobar VALUES (%d, %d, 'c3', 444)", i+i, i)) end --- diff --git a/test/sql/clear.test.lua b/test/sql/clear.test.lua index 4c58767..8d5e8fb 100644 --- a/test/sql/clear.test.lua +++ b/test/sql/clear.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -9,7 +9,7 @@ box.execute("CREATE TABLE zoobar (c1 INT, c2 INT PRIMARY KEY, c3 TEXT, c4 INT)") box.execute("CREATE UNIQUE INDEX zoobar2 ON zoobar(c1, c4)") -- Debug --- box.execute("PRAGMA vdbe_debug=ON;") +-- box.execute("SET vdbe_debug=ON;") -- Seed entry for i=1, 100 do box.execute(string.format("INSERT INTO zoobar VALUES (%d, %d, 'c3', 444)", i+i, i)) end diff --git a/test/sql/collation.result b/test/sql/collation.result index 11962ef..dc11d28 100644 --- a/test/sql/collation.result +++ b/test/sql/collation.result @@ -7,7 +7,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/collation.test.lua b/test/sql/collation.test.lua index 1be28b3..5e33562 100644 --- a/test/sql/collation.test.lua +++ b/test/sql/collation.test.lua @@ -1,7 +1,7 @@ remote = require('net.box') test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- gh-3010: COLLATE after LIMIT should throw an error diff --git a/test/sql/ddl.result b/test/sql/ddl.result index 28acf37..af833d0 100644 --- a/test/sql/ddl.result +++ b/test/sql/ddl.result @@ -8,7 +8,7 @@ json = require('json') engine = test_run:get_cfg('engine') | --- | ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') | --- | - row_count: 0 | ... diff --git a/test/sql/ddl.test.lua b/test/sql/ddl.test.lua index 6067b61..b6f0c81 100644 --- a/test/sql/ddl.test.lua +++ b/test/sql/ddl.test.lua @@ -1,7 +1,7 @@ test_run = require('test_run').new() json = require('json') engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- -- gh-4086: SQL transactional DDL. diff --git a/test/sql/delete-multiple-idx.result b/test/sql/delete-multiple-idx.result index ca58feb..3d58daf 100644 --- a/test/sql/delete-multiple-idx.result +++ b/test/sql/delete-multiple-idx.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -19,7 +19,7 @@ box.execute("CREATE UNIQUE INDEX t3y ON t3(y);"); - row_count: 1 ... -- Debug. --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries. box.execute("INSERT INTO t3 VALUES (1, 1, NULL);"); --- diff --git a/test/sql/delete-multiple-idx.test.lua b/test/sql/delete-multiple-idx.test.lua index a81cccc..2715d2f 100644 --- a/test/sql/delete-multiple-idx.test.lua +++ b/test/sql/delete-multiple-idx.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -9,7 +9,7 @@ box.execute("CREATE TABLE t3(id INT primary key,x INT,y INT);"); box.execute("CREATE UNIQUE INDEX t3y ON t3(y);"); -- Debug. --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries. box.execute("INSERT INTO t3 VALUES (1, 1, NULL);"); diff --git a/test/sql/delete.result b/test/sql/delete.result index e27c79d..9aa074f 100644 --- a/test/sql/delete.result +++ b/test/sql/delete.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -15,7 +15,7 @@ box.execute("CREATE TABLE t1(a INT, b INT, PRIMARY KEY(a, b));"); - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO t1 VALUES(1, 2);"); --- diff --git a/test/sql/delete.test.lua b/test/sql/delete.test.lua index 75448d4..ff1e622 100644 --- a/test/sql/delete.test.lua +++ b/test/sql/delete.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -8,7 +8,7 @@ box.execute('pragma sql_default_engine=\''..engine..'\'') box.execute("CREATE TABLE t1(a INT, b INT, PRIMARY KEY(a, b));"); -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO t1 VALUES(1, 2);"); diff --git a/test/sql/drop-index.result b/test/sql/drop-index.result index e8eb642..551e56f 100644 --- a/test/sql/drop-index.result +++ b/test/sql/drop-index.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -23,7 +23,7 @@ box.execute("CREATE INDEX zoobar3 ON zzoobar(c3)") - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zzoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zzoobar VALUES (111, 222, 'c3', 444)") -- Dummy entry box.execute("INSERT INTO zzoobar VALUES (111, 222, 'c3', 444)") --- diff --git a/test/sql/drop-index.test.lua b/test/sql/drop-index.test.lua index 8cd86ee..fd420b6 100644 --- a/test/sql/drop-index.test.lua +++ b/test/sql/drop-index.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -11,7 +11,7 @@ box.execute("CREATE UNIQUE INDEX zoobar2 ON zzoobar(c1, c4)") box.execute("CREATE INDEX zoobar3 ON zzoobar(c3)") -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zzoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zzoobar VALUES (111, 222, 'c3', 444)") -- Dummy entry box.execute("INSERT INTO zzoobar VALUES (111, 222, 'c3', 444)") diff --git a/test/sql/drop-table.result b/test/sql/drop-table.result index 7bc073d..ef4c5cf 100644 --- a/test/sql/drop-table.result +++ b/test/sql/drop-table.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -15,7 +15,7 @@ box.execute("CREATE TABLE zzzoobar (c1 INT, c2 INT PRIMARY KEY, c3 TEXT, c4 INT) - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zzzoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zzzoobar VALUES (111, 222, 'c3', 444)") box.execute("CREATE INDEX zb ON zzzoobar(c1, c3)") --- - row_count: 1 diff --git a/test/sql/drop-table.test.lua b/test/sql/drop-table.test.lua index a310db1..f0b1645 100644 --- a/test/sql/drop-table.test.lua +++ b/test/sql/drop-table.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -8,7 +8,7 @@ box.execute('pragma sql_default_engine=\''..engine..'\'') box.execute("CREATE TABLE zzzoobar (c1 INT, c2 INT PRIMARY KEY, c3 TEXT, c4 INT)") -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zzzoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zzzoobar VALUES (111, 222, 'c3', 444)") box.execute("CREATE INDEX zb ON zzzoobar(c1, c3)") diff --git a/test/sql/engine.result b/test/sql/engine.result index 3ee93ad..b392566 100644 --- a/test/sql/engine.result +++ b/test/sql/engine.result @@ -4,7 +4,7 @@ env = require('test_run') test_run = env.new() --- ... -box.execute("pragma sql_default_engine='vinyl'") +box.execute("set sql_default_engine='vinyl'") --- - row_count: 0 ... @@ -16,7 +16,7 @@ box.execute("CREATE TABLE t2_vinyl(a INT PRIMARY KEY, b INT, c INT);") --- - row_count: 1 ... -box.execute("pragma sql_default_engine='memtx'") +box.execute("set sql_default_engine='memtx'") --- - row_count: 0 ... @@ -66,7 +66,7 @@ assert(box.space.T1_MEMTX.engine == 'memtx') --- - true ... -box.execute("pragma sql_default_engine='vinyl'") +box.execute("set sql_default_engine='vinyl'") --- - row_count: 0 ... diff --git a/test/sql/engine.test.lua b/test/sql/engine.test.lua index 112d3d3..bc16cc0 100644 --- a/test/sql/engine.test.lua +++ b/test/sql/engine.test.lua @@ -1,11 +1,11 @@ env = require('test_run') test_run = env.new() -box.execute("pragma sql_default_engine='vinyl'") +box.execute("set sql_default_engine='vinyl'") box.execute("CREATE TABLE t1_vinyl(a INT PRIMARY KEY, b INT, c INT);") box.execute("CREATE TABLE t2_vinyl(a INT PRIMARY KEY, b INT, c INT);") -box.execute("pragma sql_default_engine='memtx'") +box.execute("set sql_default_engine='memtx'") box.execute("CREATE TABLE t3_memtx(a INT PRIMARY KEY, b INT, c INT);") assert(box.space.T1_VINYL.engine == 'vinyl') @@ -22,7 +22,7 @@ box.execute("CREATE TABLE t1_vinyl (id INT PRIMARY KEY) WITH ENGINE = 'vinyl'") assert(box.space.T1_VINYL.engine == 'vinyl') box.execute("CREATE TABLE t1_memtx (id INT PRIMARY KEY) WITH ENGINE = 'memtx'") assert(box.space.T1_MEMTX.engine == 'memtx') -box.execute("pragma sql_default_engine='vinyl'") +box.execute("set sql_default_engine='vinyl'") box.execute("CREATE TABLE t2_vinyl (id INT PRIMARY KEY) WITH ENGINE = 'vinyl'") assert(box.space.T2_VINYL.engine == 'vinyl') box.execute("CREATE TABLE t2_memtx (id INT PRIMARY KEY) WITH ENGINE = 'memtx'") diff --git a/test/sql/errinj.result b/test/sql/errinj.result index ecc194f..2c0d960 100644 --- a/test/sql/errinj.result +++ b/test/sql/errinj.result @@ -7,7 +7,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/errinj.test.lua b/test/sql/errinj.test.lua index 2b4f74a..b4f40b2 100644 --- a/test/sql/errinj.test.lua +++ b/test/sql/errinj.test.lua @@ -1,7 +1,7 @@ remote = require('net.box') test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') errinj = box.error.injection fiber = require('fiber') diff --git a/test/sql/func-recreate.result b/test/sql/func-recreate.result index 1f69664..9e14aa8 100644 --- a/test/sql/func-recreate.result +++ b/test/sql/func-recreate.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/func-recreate.test.lua b/test/sql/func-recreate.test.lua index 5496baf..561c1ef 100644 --- a/test/sql/func-recreate.test.lua +++ b/test/sql/func-recreate.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Check errors during function create process fiber = require('fiber') diff --git a/test/sql/gh-2362-select-access-rights.result b/test/sql/gh-2362-select-access-rights.result index b15b0da..f929914 100644 --- a/test/sql/gh-2362-select-access-rights.result +++ b/test/sql/gh-2362-select-access-rights.result @@ -7,7 +7,7 @@ engine = test_run:get_cfg('engine') nb = require('net.box') --- ... -box.execute("PRAGMA sql_default_engine='"..engine.."'") +box.execute("SET sql_default_engine='"..engine.."'") --- - row_count: 0 ... diff --git a/test/sql/gh-2362-select-access-rights.test.lua b/test/sql/gh-2362-select-access-rights.test.lua index f2b66b6..5666b68 100644 --- a/test/sql/gh-2362-select-access-rights.test.lua +++ b/test/sql/gh-2362-select-access-rights.test.lua @@ -2,7 +2,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') nb = require('net.box') -box.execute("PRAGMA sql_default_engine='"..engine.."'") +box.execute("SET sql_default_engine='"..engine.."'") box.execute("CREATE TABLE t1 (s1 INT PRIMARY KEY, s2 INT UNIQUE);") box.execute("CREATE TABLE t2 (s1 INT PRIMARY KEY);") box.execute("INSERT INTO t1 VALUES (1, 1);") diff --git a/test/sql/gh-2929-primary-key.result b/test/sql/gh-2929-primary-key.result index 021d037..35daaf2 100644 --- a/test/sql/gh-2929-primary-key.result +++ b/test/sql/gh-2929-primary-key.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh-2929-primary-key.test.lua b/test/sql/gh-2929-primary-key.test.lua index 9cc6cd5..f419668 100644 --- a/test/sql/gh-2929-primary-key.test.lua +++ b/test/sql/gh-2929-primary-key.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- All tables in SQL are now WITHOUT ROW ID, so if user -- tries to create table without a primary key, an appropriate error message diff --git a/test/sql/gh-2981-check-autoinc.result b/test/sql/gh-2981-check-autoinc.result index d2938cd..11fed1e 100644 --- a/test/sql/gh-2981-check-autoinc.result +++ b/test/sql/gh-2981-check-autoinc.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh-2981-check-autoinc.test.lua b/test/sql/gh-2981-check-autoinc.test.lua index 0eb8f73..afbefa0 100644 --- a/test/sql/gh-2981-check-autoinc.test.lua +++ b/test/sql/gh-2981-check-autoinc.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') box.cfg{} diff --git a/test/sql/gh-3199-no-mem-leaks.result b/test/sql/gh-3199-no-mem-leaks.result index e7ba1d2..00211ce 100644 --- a/test/sql/gh-3199-no-mem-leaks.result +++ b/test/sql/gh-3199-no-mem-leaks.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh-3199-no-mem-leaks.test.lua b/test/sql/gh-3199-no-mem-leaks.test.lua index 54a6ce5..f63bedf 100644 --- a/test/sql/gh-3199-no-mem-leaks.test.lua +++ b/test/sql/gh-3199-no-mem-leaks.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') fiber = require('fiber') -- This test checks that no leaks of region memory happens during diff --git a/test/sql/gh-3613-idx-alter-update-2.result b/test/sql/gh-3613-idx-alter-update-2.result index ff63eb2..270f961 100644 --- a/test/sql/gh-3613-idx-alter-update-2.result +++ b/test/sql/gh-3613-idx-alter-update-2.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh-3613-idx-alter-update-2.test.lua b/test/sql/gh-3613-idx-alter-update-2.test.lua index ff5b651..33730ff 100644 --- a/test/sql/gh-3613-idx-alter-update-2.test.lua +++ b/test/sql/gh-3613-idx-alter-update-2.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') box.execute('CREATE TABLE t (s1 INT PRIMARY KEY)') box.execute('CREATE INDEX i ON t (s1)') diff --git a/test/sql/gh-3613-idx-alter-update.result b/test/sql/gh-3613-idx-alter-update.result index ba323a6..34b4a1f 100644 --- a/test/sql/gh-3613-idx-alter-update.result +++ b/test/sql/gh-3613-idx-alter-update.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh-3613-idx-alter-update.test.lua b/test/sql/gh-3613-idx-alter-update.test.lua index 3027182..389a99d 100644 --- a/test/sql/gh-3613-idx-alter-update.test.lua +++ b/test/sql/gh-3613-idx-alter-update.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') box.execute('CREATE TABLE t (s1 INT PRIMARY KEY)') box.execute('CREATE INDEX i ON t (s1)') diff --git a/test/sql/gh-3888-values-blob-assert.result b/test/sql/gh-3888-values-blob-assert.result index 4b8e7ed..5691e70 100644 --- a/test/sql/gh-3888-values-blob-assert.result +++ b/test/sql/gh-3888-values-blob-assert.result @@ -10,7 +10,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh-3888-values-blob-assert.test.lua b/test/sql/gh-3888-values-blob-assert.test.lua index 0b7c385..9680aa5 100644 --- a/test/sql/gh-3888-values-blob-assert.test.lua +++ b/test/sql/gh-3888-values-blob-assert.test.lua @@ -6,7 +6,7 @@ -- test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- check 'VALUES' against typedef keywords (should fail) box.execute('VALUES(scalar)') diff --git a/test/sql/gh2141-delete-trigger-drop-table.result b/test/sql/gh2141-delete-trigger-drop-table.result index 1d373f5..a218328 100644 --- a/test/sql/gh2141-delete-trigger-drop-table.result +++ b/test/sql/gh2141-delete-trigger-drop-table.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh2141-delete-trigger-drop-table.test.lua b/test/sql/gh2141-delete-trigger-drop-table.test.lua index 4d21fd7..2d5b987 100644 --- a/test/sql/gh2141-delete-trigger-drop-table.test.lua +++ b/test/sql/gh2141-delete-trigger-drop-table.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- create space box.execute("CREATE TABLE t(id INT PRIMARY KEY)") diff --git a/test/sql/gh2251-multiple-update.result b/test/sql/gh2251-multiple-update.result index 42ebf7f..7a21cea 100644 --- a/test/sql/gh2251-multiple-update.result +++ b/test/sql/gh2251-multiple-update.result @@ -5,7 +5,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh2251-multiple-update.test.lua b/test/sql/gh2251-multiple-update.test.lua index 4d55096..eeff047 100644 --- a/test/sql/gh2251-multiple-update.test.lua +++ b/test/sql/gh2251-multiple-update.test.lua @@ -1,7 +1,7 @@ -- Regression test for #2251 test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() diff --git a/test/sql/gh2483-remote-persistency-check.result b/test/sql/gh2483-remote-persistency-check.result index d69fcbd..a7dd7d2 100644 --- a/test/sql/gh2483-remote-persistency-check.result +++ b/test/sql/gh2483-remote-persistency-check.result @@ -8,7 +8,7 @@ test_run = env.new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh2483-remote-persistency-check.test.lua b/test/sql/gh2483-remote-persistency-check.test.lua index 7db1602..bf2fc6b 100644 --- a/test/sql/gh2483-remote-persistency-check.test.lua +++ b/test/sql/gh2483-remote-persistency-check.test.lua @@ -2,7 +2,7 @@ env = require('test_run') test_run = env.new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') box.schema.user.grant('guest', 'read,write,execute', 'universe') diff --git a/test/sql/gh2808-inline-unique-persistency-check.result b/test/sql/gh2808-inline-unique-persistency-check.result index a277b3f..db03feb 100644 --- a/test/sql/gh2808-inline-unique-persistency-check.result +++ b/test/sql/gh2808-inline-unique-persistency-check.result @@ -8,7 +8,7 @@ test_run = env.new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/gh2808-inline-unique-persistency-check.test.lua b/test/sql/gh2808-inline-unique-persistency-check.test.lua index 26b646a..ef38ae3 100644 --- a/test/sql/gh2808-inline-unique-persistency-check.test.lua +++ b/test/sql/gh2808-inline-unique-persistency-check.test.lua @@ -2,7 +2,7 @@ env = require('test_run') test_run = env.new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Create a table and insert a datum box.execute([[CREATE TABLE t1(a INT PRIMARY KEY, b INT, UNIQUE(b));]]) diff --git a/test/sql/icu-upper-lower.result b/test/sql/icu-upper-lower.result index 8ff7528..cb687a1 100644 --- a/test/sql/icu-upper-lower.result +++ b/test/sql/icu-upper-lower.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/icu-upper-lower.test.lua b/test/sql/icu-upper-lower.test.lua index 00e9699..bbd4f00 100644 --- a/test/sql/icu-upper-lower.test.lua +++ b/test/sql/icu-upper-lower.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') test_run:cmd("setopt delimiter ';'") diff --git a/test/sql/insert-unique.result b/test/sql/insert-unique.result index 1cf44c9..ec57681 100644 --- a/test/sql/insert-unique.result +++ b/test/sql/insert-unique.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -19,7 +19,7 @@ box.execute("CREATE UNIQUE INDEX zoobar2 ON zoobar(c1, c4)") - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entry box.execute("INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") --- diff --git a/test/sql/insert-unique.test.lua b/test/sql/insert-unique.test.lua index 026bc9d..243333e 100644 --- a/test/sql/insert-unique.test.lua +++ b/test/sql/insert-unique.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -9,7 +9,7 @@ box.execute("CREATE TABLE zoobar (c1 INT, c2 INT PRIMARY KEY, c3 TEXT, c4 INT)") box.execute("CREATE UNIQUE INDEX zoobar2 ON zoobar(c1, c4)") -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entry box.execute("INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") diff --git a/test/sql/integer-overflow.result b/test/sql/integer-overflow.result index 223ba02..c886f8c 100644 --- a/test/sql/integer-overflow.result +++ b/test/sql/integer-overflow.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/integer-overflow.test.lua b/test/sql/integer-overflow.test.lua index 1b3e8ce..57929e2 100644 --- a/test/sql/integer-overflow.test.lua +++ b/test/sql/integer-overflow.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- gh-3735: make sure that integer overflows errors are -- handled during VDBE execution. diff --git a/test/sql/iproto.result b/test/sql/iproto.result index 1e5c30a..bac1334 100644 --- a/test/sql/iproto.result +++ b/test/sql/iproto.result @@ -7,7 +7,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -732,37 +732,6 @@ res.metadata - name: detail type: TEXT ... --- When pragma count_changes is on, statements INSERT, REPLACE and --- UPDATE returns number of changed columns. Make sure that this --- result has a column type. -cn:execute("PRAGMA count_changes = 1;") ---- -- row_count: 0 -... -cn:execute("INSERT INTO t1 VALUES (1), (2), (3);") ---- -- metadata: - - name: rows inserted - type: INTEGER - rows: - - [3] -... -cn:execute("REPLACE INTO t1 VALUES (2), (3), (4), (5);") ---- -- metadata: - - name: rows replaced - type: INTEGER - rows: - - [4] -... -cn:execute("UPDATE t1 SET id = id + 100 WHERE id > 10;") ---- -- metadata: - - name: rows updated - type: INTEGER - rows: - - [0] -... -- Make sure that built-in functions have a right returning type. -- cn:execute("SELECT zeroblob(1);") diff --git a/test/sql/iproto.test.lua b/test/sql/iproto.test.lua index 5dfe95c..d31fb7f 100644 --- a/test/sql/iproto.test.lua +++ b/test/sql/iproto.test.lua @@ -1,7 +1,7 @@ remote = require('net.box') test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') box.execute('create table test (id int primary key, a NUMBER, b text)') space = box.space.TEST @@ -220,14 +220,6 @@ res.metadata res = cn:execute("EXPLAIN QUERY PLAN SELECT COUNT(*) FROM t1") res.metadata --- When pragma count_changes is on, statements INSERT, REPLACE and --- UPDATE returns number of changed columns. Make sure that this --- result has a column type. -cn:execute("PRAGMA count_changes = 1;") -cn:execute("INSERT INTO t1 VALUES (1), (2), (3);") -cn:execute("REPLACE INTO t1 VALUES (2), (3), (4), (5);") -cn:execute("UPDATE t1 SET id = id + 100 WHERE id > 10;") - -- Make sure that built-in functions have a right returning type. -- cn:execute("SELECT zeroblob(1);") diff --git a/test/sql/max-on-index.result b/test/sql/max-on-index.result index 57ce95b..45f29fd 100644 --- a/test/sql/max-on-index.result +++ b/test/sql/max-on-index.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -25,7 +25,7 @@ box.execute("CREATE TABLE test2 (f1 INT, f2 INT, PRIMARY KEY(f1))") - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO test1 VALUES(1, 2)"); --- diff --git a/test/sql/max-on-index.test.lua b/test/sql/max-on-index.test.lua index 4cceaa7..73e4199 100644 --- a/test/sql/max-on-index.test.lua +++ b/test/sql/max-on-index.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -13,7 +13,7 @@ box.execute("CREATE INDEX test1_index ON test1 (f2)") box.execute("CREATE TABLE test2 (f1 INT, f2 INT, PRIMARY KEY(f1))") -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO test1 VALUES(1, 2)"); diff --git a/test/sql/message-func-indexes.result b/test/sql/message-func-indexes.result index 69e3ee0..d198e14 100644 --- a/test/sql/message-func-indexes.result +++ b/test/sql/message-func-indexes.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/message-func-indexes.test.lua b/test/sql/message-func-indexes.test.lua index 9ac5f47..dc67606 100644 --- a/test/sql/message-func-indexes.test.lua +++ b/test/sql/message-func-indexes.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Creating tables. box.execute("CREATE TABLE t1(id INTEGER PRIMARY KEY, a INTEGER)") diff --git a/test/sql/misc.result b/test/sql/misc.result index a157ddb..b06ac57 100644 --- a/test/sql/misc.result +++ b/test/sql/misc.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/misc.test.lua b/test/sql/misc.test.lua index 541660c..e581b73 100644 --- a/test/sql/misc.test.lua +++ b/test/sql/misc.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Forbid multistatement queries. box.execute('select 1;') diff --git a/test/sql/no-pk-space.result b/test/sql/no-pk-space.result index 025f363..d0274ea 100644 --- a/test/sql/no-pk-space.result +++ b/test/sql/no-pk-space.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/no-pk-space.test.lua b/test/sql/no-pk-space.test.lua index 318c2ac..f451b57 100644 --- a/test/sql/no-pk-space.test.lua +++ b/test/sql/no-pk-space.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') format = {} format[1] = {'id', 'integer'} diff --git a/test/sql/on-conflict.result b/test/sql/on-conflict.result index 6851e21..c0bb802 100644 --- a/test/sql/on-conflict.result +++ b/test/sql/on-conflict.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/on-conflict.test.lua b/test/sql/on-conflict.test.lua index 1aa4d1b..958303e 100644 --- a/test/sql/on-conflict.test.lua +++ b/test/sql/on-conflict.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- -- Check that original sql ON CONFLICT clause is really -- disabled. diff --git a/test/sql/persistency.result b/test/sql/persistency.result index f8f992c..88778ec 100644 --- a/test/sql/persistency.result +++ b/test/sql/persistency.result @@ -7,7 +7,7 @@ test_run = env.new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/persistency.test.lua b/test/sql/persistency.test.lua index 1964453..72d1992 100644 --- a/test/sql/persistency.test.lua +++ b/test/sql/persistency.test.lua @@ -1,7 +1,7 @@ env = require('test_run') test_run = env.new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- create space box.execute("CREATE TABLE foobar (foo INT PRIMARY KEY, bar TEXT)") diff --git a/test/sql/row-count.result b/test/sql/row-count.result index fb96e21..eb0114b 100644 --- a/test/sql/row-count.result +++ b/test/sql/row-count.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -314,13 +314,9 @@ box.execute("SELECT ROW_COUNT();") rows: - [0] ... -box.execute('PRAGMA recursive_triggers') +box.execute("SET sql_default_engine='"..engine.."';") --- -- metadata: - - name: recursive_triggers - type: INTEGER - rows: - - [1] +- row_count: 0 ... -- Clean-up. -- diff --git a/test/sql/row-count.test.lua b/test/sql/row-count.test.lua index 369e7fa..06686ae 100644 --- a/test/sql/row-count.test.lua +++ b/test/sql/row-count.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Test cases concerning row count calculations. -- @@ -65,7 +65,7 @@ box.execute("SELECT ROW_COUNT();") box.execute("SELECT ROW_COUNT();") box.execute("EXPLAIN QUERY PLAN INSERT INTO t1 VALUES ('b'), ('c'), ('d');") box.execute("SELECT ROW_COUNT();") -box.execute('PRAGMA recursive_triggers') +box.execute("SET sql_default_engine='"..engine.."';") -- Clean-up. -- diff --git a/test/sql/savepoints.result b/test/sql/savepoints.result index e48db30..a111e26 100644 --- a/test/sql/savepoints.result +++ b/test/sql/savepoints.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/savepoints.test.lua b/test/sql/savepoints.test.lua index 99622a4..0af67a9 100644 --- a/test/sql/savepoints.test.lua +++ b/test/sql/savepoints.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- These tests check that SQL savepoints properly work outside -- transactions as well as inside transactions started in Lua. diff --git a/test/sql/select-null.result b/test/sql/select-null.result index 83d9776..bd25f03 100644 --- a/test/sql/select-null.result +++ b/test/sql/select-null.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -15,7 +15,7 @@ box.execute("CREATE TABLE t3(id INT, a text, b TEXT, PRIMARY KEY(id))") - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO t3 VALUES(1, 'abc',NULL)"); --- diff --git a/test/sql/select-null.test.lua b/test/sql/select-null.test.lua index a49eb43..9f54048 100644 --- a/test/sql/select-null.test.lua +++ b/test/sql/select-null.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -8,7 +8,7 @@ box.execute('pragma sql_default_engine=\''..engine..'\'') box.execute("CREATE TABLE t3(id INT, a text, b TEXT, PRIMARY KEY(id))") -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO t3 VALUES(1, 'abc',NULL)"); diff --git a/test/sql/sql-debug.result b/test/sql/sql-debug.result index 07542e3..61970b0 100644 --- a/test/sql/sql-debug.result +++ b/test/sql/sql-debug.result @@ -5,56 +5,6 @@ test_run = require('test_run').new() --- ... -- --- gh-3832: Some statements do not return column type --- Check that "PRAGMA parser_trace" returns 0 or 1 if called --- without parameter. -result = box.execute('PRAGMA parser_trace').rows ---- -... -box.execute('PRAGMA parser_trace = 1') ---- -- row_count: 0 -... -box.execute('PRAGMA parser_trace') ---- -- metadata: - - name: parser_trace - type: INTEGER - rows: - - [1] -... -box.execute('PRAGMA parser_trace = '.. result[1][1]) ---- -- row_count: 0 -... --- --- Make PRAGMA command return the result as a result set. --- -box.execute('PRAGMA') ---- -- metadata: - - name: pragma_name - type: TEXT - - name: pragma_value - type: INTEGER - rows: - - ['count_changes', 0] - - ['defer_foreign_keys', 0] - - ['full_column_names', 0] - - ['parser_trace', 0] - - ['recursive_triggers', 1] - - ['reverse_unordered_selects', 0] - - ['select_trace', 0] - - ['short_column_names', 1] - - ['sql_trace', 0] - - ['vdbe_addoptrace', 0] - - ['vdbe_debug', 0] - - ['vdbe_eqp', 0] - - ['vdbe_listing', 0] - - ['vdbe_trace', 0] - - ['where_trace', 0] -... --- -- gh-4511: make sure that SET works. -- box.execute('SELECT "name" FROM "_vsql_settings";') diff --git a/test/sql/sql-debug.test.lua b/test/sql/sql-debug.test.lua index 60d7fdd..c38ca43 100644 --- a/test/sql/sql-debug.test.lua +++ b/test/sql/sql-debug.test.lua @@ -2,21 +2,6 @@ remote = require('net.box') test_run = require('test_run').new() -- --- gh-3832: Some statements do not return column type - --- Check that "PRAGMA parser_trace" returns 0 or 1 if called --- without parameter. -result = box.execute('PRAGMA parser_trace').rows -box.execute('PRAGMA parser_trace = 1') -box.execute('PRAGMA parser_trace') -box.execute('PRAGMA parser_trace = '.. result[1][1]) - --- --- Make PRAGMA command return the result as a result set. --- -box.execute('PRAGMA') - --- -- gh-4511: make sure that SET works. -- box.execute('SELECT "name" FROM "_vsql_settings";') diff --git a/test/sql/sql-statN-index-drop.result b/test/sql/sql-statN-index-drop.result index b2a4458..cf0f6f4 100644 --- a/test/sql/sql-statN-index-drop.result +++ b/test/sql/sql-statN-index-drop.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/sql-statN-index-drop.test.lua b/test/sql/sql-statN-index-drop.test.lua index 5477a2a..5f9cc68 100644 --- a/test/sql/sql-statN-index-drop.test.lua +++ b/test/sql/sql-statN-index-drop.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Initializing some things. box.execute("CREATE TABLE t1(id INT PRIMARY KEY, a INT);") diff --git a/test/sql/tokenizer.result b/test/sql/tokenizer.result index 1ae9ef2..07a7c41 100644 --- a/test/sql/tokenizer.result +++ b/test/sql/tokenizer.result @@ -7,7 +7,7 @@ test_run = env.new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/tokenizer.test.lua b/test/sql/tokenizer.test.lua index 3f5dd12..15cf658 100644 --- a/test/sql/tokenizer.test.lua +++ b/test/sql/tokenizer.test.lua @@ -1,7 +1,7 @@ env = require('test_run') test_run = env.new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') sql_tokenizer = require('sql_tokenizer') diff --git a/test/sql/transition.result b/test/sql/transition.result index 9738092..35e7af6 100644 --- a/test/sql/transition.result +++ b/test/sql/transition.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/transition.test.lua b/test/sql/transition.test.lua index a05c26a..9b8b868 100644 --- a/test/sql/transition.test.lua +++ b/test/sql/transition.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- create space box.execute("CREATE TABLE foobar (foo INT PRIMARY KEY, bar TEXT)") diff --git a/test/sql/transitive-transactions.result b/test/sql/transitive-transactions.result index 29c7316..9308f0b 100644 --- a/test/sql/transitive-transactions.result +++ b/test/sql/transitive-transactions.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute("pragma sql_default_engine=\'"..engine.."\'") +box.execute("set sql_default_engine=\'"..engine.."\'") --- - row_count: 0 ... @@ -88,7 +88,7 @@ box.space.PARENT:select(); --- - - [1, 1] ... --- Make sure that 'PRAGMA defer_foreign_keys' works. +-- Make sure that SQL option 'defer_foreign_keys' works. -- box.execute('DROP TABLE child;') box.execute('CREATE TABLE child(id INT PRIMARY KEY, x INT REFERENCES parent(y))') @@ -116,7 +116,7 @@ box.space.PARENT:select(); --- - - [1, 1] ... -box.execute('PRAGMA defer_foreign_keys = 1;') +box.execute('SET defer_foreign_keys = true;') box.rollback() fk_defer(); --- @@ -131,7 +131,7 @@ box.space.PARENT:select(); - [2, 2] ... -- Cleanup -box.execute('PRAGMA defer_foreign_keys = 0;') +box.execute('SET defer_foreign_keys = false;') box.execute('DROP TABLE child;'); --- diff --git a/test/sql/transitive-transactions.test.lua b/test/sql/transitive-transactions.test.lua index 4633f07..f7c8b6e 100644 --- a/test/sql/transitive-transactions.test.lua +++ b/test/sql/transitive-transactions.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute("pragma sql_default_engine=\'"..engine.."\'") +box.execute("set sql_default_engine=\'"..engine.."\'") test_run:cmd("setopt delimiter ';'") -- These tests are aimed at checking transitive transactions @@ -45,7 +45,7 @@ fk_violation_3(); box.space.CHILD:select(); box.space.PARENT:select(); --- Make sure that 'PRAGMA defer_foreign_keys' works. +-- Make sure that SQL option 'defer_foreign_keys' works. -- box.execute('DROP TABLE child;') box.execute('CREATE TABLE child(id INT PRIMARY KEY, x INT REFERENCES parent(y))') @@ -62,13 +62,13 @@ end; fk_defer(); box.space.CHILD:select(); box.space.PARENT:select(); -box.execute('PRAGMA defer_foreign_keys = 1;') +box.execute('SET defer_foreign_keys = true;') box.rollback() fk_defer(); box.space.CHILD:select(); box.space.PARENT:select(); -box.execute('PRAGMA defer_foreign_keys = 0;') +box.execute('SET defer_foreign_keys = false;') -- Cleanup box.execute('DROP TABLE child;'); diff --git a/test/sql/triggers.result b/test/sql/triggers.result index 9dfe981..4b86afb 100644 --- a/test/sql/triggers.result +++ b/test/sql/triggers.result @@ -7,7 +7,7 @@ test_run = env.new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -292,7 +292,7 @@ box.execute("DROP TABLE T1;") -- gh-3531: Assertion with trigger and two storage engines -- -- Case 1: Src 'vinyl' table; Dst 'memtx' table -box.execute("PRAGMA sql_default_engine ('vinyl');") +box.execute("SET sql_default_engine ('vinyl');") --- - row_count: 0 ... @@ -304,7 +304,7 @@ box.execute("CREATE TRIGGER m1 BEFORE UPDATE ON m FOR EACH ROW BEGIN UPDATE n SE --- - row_count: 1 ... -box.execute("PRAGMA sql_default_engine('memtx');") +box.execute("SET sql_default_engine('memtx');") --- - row_count: 0 ... @@ -336,7 +336,7 @@ box.execute("DROP TABLE n;") - row_count: 1 ... -- Case 2: Src 'memtx' table; Dst 'vinyl' table -box.execute("PRAGMA sql_default_engine ('memtx');") +box.execute("SET sql_default_engine ('memtx');") --- - row_count: 0 ... @@ -348,7 +348,7 @@ box.execute("CREATE TRIGGER m1 BEFORE UPDATE ON m FOR EACH ROW BEGIN UPDATE n SE --- - row_count: 1 ... -box.execute("PRAGMA sql_default_engine('vinyl');") +box.execute("SET sql_default_engine('vinyl');") --- - row_count: 0 ... @@ -380,7 +380,7 @@ box.execute("DROP TABLE n;") - row_count: 1 ... -- Test SQL Transaction with LUA -box.execute("PRAGMA sql_default_engine ('memtx');") +box.execute("SET sql_default_engine ('memtx');") --- - row_count: 0 ... @@ -388,7 +388,7 @@ box.execute("CREATE TABLE test (id INT PRIMARY KEY)") --- - row_count: 1 ... -box.execute("PRAGMA sql_default_engine='vinyl'") +box.execute("SET sql_default_engine='vinyl'") --- - row_count: 0 ... diff --git a/test/sql/triggers.test.lua b/test/sql/triggers.test.lua index f0397dc..62f19d8 100644 --- a/test/sql/triggers.test.lua +++ b/test/sql/triggers.test.lua @@ -1,7 +1,7 @@ env = require('test_run') test_run = env.new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Get invariant part of the tuple; name and opts don't change. function immutable_part(data) local r = {} for i, l in pairs(data) do table.insert(r, {l.name, l.opts}) end return r end @@ -100,10 +100,10 @@ box.execute("DROP TABLE T1;") -- gh-3531: Assertion with trigger and two storage engines -- -- Case 1: Src 'vinyl' table; Dst 'memtx' table -box.execute("PRAGMA sql_default_engine ('vinyl');") +box.execute("SET sql_default_engine ('vinyl');") box.execute("CREATE TABLE m (s0 INT PRIMARY KEY, s1 TEXT UNIQUE);") box.execute("CREATE TRIGGER m1 BEFORE UPDATE ON m FOR EACH ROW BEGIN UPDATE n SET s2 = 'now'; END;") -box.execute("PRAGMA sql_default_engine('memtx');") +box.execute("SET sql_default_engine('memtx');") box.execute("CREATE TABLE n (s0 INT PRIMARY KEY, s1 TEXT UNIQUE, s2 NUMBER);") box.execute("INSERT INTO m VALUES (0, '0');") box.execute("INSERT INTO n VALUES (0, '',null);") @@ -116,10 +116,10 @@ box.execute("DROP TABLE n;") -- Case 2: Src 'memtx' table; Dst 'vinyl' table -box.execute("PRAGMA sql_default_engine ('memtx');") +box.execute("SET sql_default_engine ('memtx');") box.execute("CREATE TABLE m (s0 INT PRIMARY KEY, s1 TEXT UNIQUE);") box.execute("CREATE TRIGGER m1 BEFORE UPDATE ON m FOR EACH ROW BEGIN UPDATE n SET s2 = 'now'; END;") -box.execute("PRAGMA sql_default_engine('vinyl');") +box.execute("SET sql_default_engine('vinyl');") box.execute("CREATE TABLE n (s0 INT PRIMARY KEY, s1 TEXT UNIQUE, s2 NUMBER);") box.execute("INSERT INTO m VALUES (0, '0');") box.execute("INSERT INTO n VALUES (0, '',null);") @@ -131,9 +131,9 @@ box.execute("DROP TABLE m;") box.execute("DROP TABLE n;") -- Test SQL Transaction with LUA -box.execute("PRAGMA sql_default_engine ('memtx');") +box.execute("SET sql_default_engine ('memtx');") box.execute("CREATE TABLE test (id INT PRIMARY KEY)") -box.execute("PRAGMA sql_default_engine='vinyl'") +box.execute("SET sql_default_engine='vinyl'") box.execute("CREATE TABLE test2 (id INT PRIMARY KEY)") box.execute("INSERT INTO test2 VALUES (2)") box.execute("START TRANSACTION") diff --git a/test/sql/update-with-nested-select.result b/test/sql/update-with-nested-select.result index 3172430..b6ccda2 100644 --- a/test/sql/update-with-nested-select.result +++ b/test/sql/update-with-nested-select.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... @@ -15,7 +15,7 @@ box.execute("CREATE TABLE t1(a integer primary key, b INT UNIQUE, e INT);"); - row_count: 1 ... -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO t1 VALUES(1,4,6);"); --- diff --git a/test/sql/update-with-nested-select.test.lua b/test/sql/update-with-nested-select.test.lua index 88424fc..07587ff 100644 --- a/test/sql/update-with-nested-select.test.lua +++ b/test/sql/update-with-nested-select.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- box.cfg() @@ -8,7 +8,7 @@ box.execute('pragma sql_default_engine=\''..engine..'\'') box.execute("CREATE TABLE t1(a integer primary key, b INT UNIQUE, e INT);"); -- Debug --- box.execute("PRAGMA vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") +-- box.execute("SET vdbe_debug=ON ; INSERT INTO zoobar VALUES (111, 222, 'c3', 444)") -- Seed entries box.execute("INSERT INTO t1 VALUES(1,4,6);"); diff --git a/test/sql/upgrade.result b/test/sql/upgrade.result index f0997e1..6627777 100644 --- a/test/sql/upgrade.result +++ b/test/sql/upgrade.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/upgrade.test.lua b/test/sql/upgrade.test.lua index 37425ae..0c882ba 100644 --- a/test/sql/upgrade.test.lua +++ b/test/sql/upgrade.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') work_dir = 'sql/upgrade/1.10/' test_run:cmd('create server upgrade with script="sql/upgrade/upgrade.lua", workdir="' .. work_dir .. '"') diff --git a/test/sql/view.result b/test/sql/view.result index d845df8..3df08fc 100644 --- a/test/sql/view.result +++ b/test/sql/view.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/view.test.lua b/test/sql/view.test.lua index 0008056..6a1fae2 100644 --- a/test/sql/view.test.lua +++ b/test/sql/view.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') -- Verify that constraints on 'view' option are working. diff --git a/test/sql/view_delayed_wal.result b/test/sql/view_delayed_wal.result index d518e7d..a5f64f2 100644 --- a/test/sql/view_delayed_wal.result +++ b/test/sql/view_delayed_wal.result @@ -4,7 +4,7 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') --- ... -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') --- - row_count: 0 ... diff --git a/test/sql/view_delayed_wal.test.lua b/test/sql/view_delayed_wal.test.lua index 7e6fce6..ec93e72 100644 --- a/test/sql/view_delayed_wal.test.lua +++ b/test/sql/view_delayed_wal.test.lua @@ -1,6 +1,6 @@ test_run = require('test_run').new() engine = test_run:get_cfg('engine') -box.execute('pragma sql_default_engine=\''..engine..'\'') +box.execute('set sql_default_engine=\''..engine..'\'') fiber = require('fiber') -- View reference counters are incremented before firing diff --git a/test/sql/vinyl-opts.result b/test/sql/vinyl-opts.result index 10a649a..b9e07c1 100644 --- a/test/sql/vinyl-opts.result +++ b/test/sql/vinyl-opts.result @@ -13,7 +13,7 @@ test_run:cmd("switch test") --- - true ... -box.execute('pragma sql_default_engine= \'vinyl\'') +box.execute('set sql_default_engine= \'vinyl\'') --- - row_count: 0 ... diff --git a/test/sql/vinyl-opts.test.lua b/test/sql/vinyl-opts.test.lua index 4460724..05864d0 100644 --- a/test/sql/vinyl-opts.test.lua +++ b/test/sql/vinyl-opts.test.lua @@ -3,7 +3,7 @@ test_run:cmd("create server test with script='sql/vinyl-opts-cfg.lua'") test_run:cmd("start server test") test_run:cmd("switch test") -box.execute('pragma sql_default_engine= \'vinyl\'') +box.execute('set sql_default_engine= \'vinyl\'') box.execute('CREATE TABLE v1 (id INT PRIMARY KEY, b INT);') box.space.V1.index[0].options -- 2.7.4
next prev parent reply other threads:[~2019-10-17 14:40 UTC|newest] Thread overview: 9+ messages / expand[flat|nested] mbox.gz Atom feed top 2019-10-17 14:40 [Tarantool-patches] [PATCH v1 0/5] Replace control pragmas by SET imeevma 2019-10-17 14:40 ` [Tarantool-patches] [PATCH v1 1/5] sysview: make get() and create_iterator() methods virtual imeevma 2019-10-17 14:40 ` [Tarantool-patches] [PATCH v1 2/5] box: introduce _vsql_settings sysview imeevma 2019-10-18 22:08 ` [Tarantool-patches] [tarantool-patches] " Vladislav Shpilevoy 2019-10-17 14:40 ` [Tarantool-patches] [PATCH v1 3/5] sql: create SET command imeevma 2019-10-18 22:08 ` [Tarantool-patches] [tarantool-patches] " Vladislav Shpilevoy 2019-10-17 14:40 ` [Tarantool-patches] [PATCH v1 4/5] temporary: disable boolean.test.sql imeevma 2019-10-17 14:40 ` imeevma [this message] 2019-10-18 22:08 ` [Tarantool-patches] [tarantool-patches] [PATCH v1 0/5] Replace control pragmas by SET Vladislav Shpilevoy
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=40c780065c6746ac8891732dc5bb0e16938421cc.1571322776.git.imeevma@gmail.com \ --to=imeevma@tarantool.org \ --cc=tarantool-patches@dev.tarantool.org \ --cc=tarantool-patches@freelists.org \ --cc=v.shpilevoy@tarantool.org \ --subject='Re: [Tarantool-patches] [PATCH v1 5/5] sql: replace control pragmas' \ /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