[tarantool-patches] Re: [PATCH v2 1/3] sql: remove unnecessary AUTOINCREMENT ID generation
n.pettik
korablev at tarantool.org
Mon Jul 8 21:54:11 MSK 2019
> On 4 Jul 2019, at 14:42, imeevma at tarantool.org wrote:
>
> Currently, if we perform something like
> CREATE TABLE t (i INT PRIMARY KEY AUTOINCREMENT);
> INSERT INTO t(a) VALUES (NULL);
>
> we generate a new identifier in a special way.
Describe this “special” way.
> src/box/sql/insert.c | 5 +----
> test/sql/gh-2981-check-autoinc.result | 32 ++++++++++++++++++++++++++++++++
> test/sql/gh-2981-check-autoinc.test.lua | 11 ++++++++++-
> 3 files changed, 43 insertions(+), 5 deletions(-)
>
> diff --git a/src/box/sql/insert.c b/src/box/sql/insert.c
> index b353148..d2b4e17 100644
> --- a/src/box/sql/insert.c
> +++ b/src/box/sql/insert.c
> @@ -628,10 +628,7 @@ sqlInsert(Parse * pParse, /* Parser context */
> if (j < 0 || nColumn == 0
> || (pColumn && j >= pColumn->nId)) {
> if (i == (int) autoinc_fieldno) {
> - sqlVdbeAddOp2(v,
> - OP_NextAutoincValue,
> - space->def->id,
> - iRegStore);
Why didn’t you delete OP_NextAutioncValue?
> + sqlVdbeAddOp2(v, OP_Null, 0, iRegStore);
> continue;
> }
> struct Expr *dflt = NULL;
>
> diff --git a/test/sql/gh-2981-check-autoinc.test.lua b/test/sql/gh-2981-check-autoinc.test.lua
> index 0eb8f73..ac5624e 100644
> --- a/test/sql/gh-2981-check-autoinc.test.lua
> +++ b/test/sql/gh-2981-check-autoinc.test.lua
> @@ -7,6 +7,8 @@ box.cfg{}
> box.execute("CREATE TABLE t1 (s1 INTEGER PRIMARY KEY AUTOINCREMENT, s2 INTEGER, CHECK (s1 <> 19));");
> box.execute("CREATE TABLE t2 (s1 INTEGER PRIMARY KEY AUTOINCREMENT, s2 INTEGER, CHECK (s1 <> 19 AND s1 <> 25));");
> box.execute("CREATE TABLE t3 (s1 INTEGER PRIMARY KEY AUTOINCREMENT, s2 INTEGER, CHECK (s1 < 10));”);
> +box.execute("CREATE TABLE t4 (s1 INTEGER PRIMARY KEY AUTOINCREMENT, s2 INTEGER, CHECK (s1 < 10));");
> +box.execute("CREATE TABLE t5 (s1 INTEGER PRIMARY KEY AUTOINCREMENT, s2 INTEGER, CHECK (s1 <> 19));”);
Add explanation to these tests.. What do they verify?
I build master branch and they are passed as well.
> box.execute("insert into t1 values (18, null);")
> box.execute("insert into t1(s2) values (null);")
> @@ -19,7 +21,14 @@ box.execute("insert into t2(s2) values (null);")
> box.execute("insert into t3 values (9, null)")
> box.execute("insert into t3(s2) values (null)")
>
> +box.execute("insert into t4 values (9, null)")
> +box.execute("insert into t4 values (null, null)")
> +
> +box.execute("INSERT INTO t5 VALUES (18, NULL);")
> +box.execute("INSERT INTO t5 SELECT NULL, NULL FROM t5;")
> +
More information about the Tarantool-patches
mailing list