From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from [87.239.111.99] (localhost [127.0.0.1]) by dev.tarantool.org (Postfix) with ESMTP id BB3D56ECC0; Wed, 15 Dec 2021 13:58:37 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org BB3D56ECC0 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=tarantool.org; s=dev; t=1639565917; bh=GbYp839NoEUC6aSMbzWcJ9EQqZfpPYqCr2W2J82n9ls=; h=To:Cc:Date:Subject:List-Id:List-Unsubscribe:List-Archive: List-Post:List-Help:List-Subscribe:From:Reply-To:From; b=lfjDEquMepZpDnbo7+iPRL8oUK9EGxHNlMFnpYhbfryQRw062Ue0E1gW+jwrpVoX4 z4XLfx+8EZaRCU1/RxqlhDfHqEFDQhkdHnj+81xbKJ7BsIoYjMqdUBHU4wzyXn5zWG ZgPxkedm2uaQDduMTIS1np+bsfbc9VsDZ+pQxrsM= Received: from smtpng1.i.mail.ru (smtpng1.i.mail.ru [94.100.181.251]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by dev.tarantool.org (Postfix) with ESMTPS id AD2B46ECC0 for ; Wed, 15 Dec 2021 13:58:35 +0300 (MSK) DKIM-Filter: OpenDKIM Filter v2.11.0 dev.tarantool.org AD2B46ECC0 Received: by smtpng1.m.smailru.net with esmtpa (envelope-from ) id 1mxRzW-0000GT-Lb; Wed, 15 Dec 2021 13:58:35 +0300 To: kyukhin@tarantool.org Cc: tarantool-patches@dev.tarantool.org Date: Wed, 15 Dec 2021 13:58:34 +0300 Message-Id: <4b3955f0797e12b2946b175282cc7b8f9955cc26.1639565874.git.imeevma@gmail.com> X-Mailer: git-send-email 2.25.1 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-7564579A: 646B95376F6C166E X-77F55803: 4F1203BC0FB41BD9FFF274446F725B74506C72DDA7EBE5A636C5C70934101B1A182A05F538085040B39B274AD62353FB4DC2ADCCB6C57C50361A10600B4C81D9666D34C4C271D569 X-7FA49CB5: FF5795518A3D127A4AD6D5ED66289B5278DA827A17800CE7EED5D2FAB4CEB1EDEA1F7E6F0F101C67BD4B6F7A4D31EC0BCC500DACC3FED6E28638F802B75D45FF8AA50765F7900637E893C22CB255350D8638F802B75D45FF36EB9D2243A4F8B5A6FCA7DBDB1FC311F39EFFDF887939037866D6147AF826D816736DAD6BA49419D583C660AD529168117882F4460429724CE54428C33FAD305F5C1EE8F4F765FCAA867293B0326636D2E47CDBA5A96583BD4B6F7A4D31EC0BC014FD901B82EE079FA2833FD35BB23D27C277FBC8AE2E8BAA867293B0326636D2E47CDBA5A96583BA9C0B312567BB231DD303D21008E29813377AFFFEAFD269A417C69337E82CC2E827F84554CEF50127C277FBC8AE2E8BA83251EDC214901ED5E8D9A59859A8B6A45692FFBBD75A6A089D37D7C0E48F6C5571747095F342E88FB05168BE4CE3AF X-C1DE0DAB: C20DE7B7AB408E4181F030C43753B8186998911F362727C414F749A5E30D975C302CFC0D3F89A23B753C8D54CE7EA40A385E5F68AB72F1899C2B6934AE262D3EE7EAB7254005DCED42F5E56778F0CDB29510FB958DCE06DB6ED91DBE5ABE359A7EE5648E065588D469F8FEF10F1C2C2993EDB24507CE13387DFF0A840B692CF8 X-C8649E89: 4E36BF7865823D7055A7F0CF078B5EC49A30900B95165D34B7CBFF60649FF266942B822D8DDA40B5F7F8AFA33DA32B6E3364A87A9BD38B546CEB4AC071C818F41D7E09C32AA3244CF1A984104C20BC90878A024C447F6DBD5A1673A01BA68E40729B2BEF169E0186 X-D57D3AED: 3ZO7eAau8CL7WIMRKs4sN3D3tLDjz0dLbV79QFUyzQ2Ujvy7cMT6pYYqY16iZVKkSc3dCLJ7zSJH7+u4VD18S7Vl4ZUrpaVfd2+vE6kuoey4m4VkSEu530nj6fImhcD4MUrOEAnl0W826KZ9Q+tr5ycPtXkTV4k65bRjmOUUP8cvGozZ33TWg5HZplvhhXbhDGzqmQDTd6OAevLeAnq3Ra9uf7zvY2zzsIhlcp/Y7m53TZgf2aB4JOg4gkr2biojbL9S8ysBdXgONM3geEhExk8N7LSelQs7 X-Mailru-Sender: 689FA8AB762F7393C37E3C1AEC41BA5DDF3965DDD096AF59A5E641022763FB5C83D72C36FC87018B9F80AB2734326CD2FB559BB5D741EB96352A0ABBE4FDA4210A04DAD6CC59E3365FEEDEB644C299C0ED14614B50AE0675 X-Mras: Ok Subject: [Tarantool-patches] [PATCH v1 1/1] sql: introduce binding for MAP X-BeenThere: tarantool-patches@dev.tarantool.org X-Mailman-Version: 2.1.34 Precedence: list List-Id: Tarantool development patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Mergen Imeev via Tarantool-patches Reply-To: imeevma@tarantool.org Errors-To: tarantool-patches-bounces@dev.tarantool.org Sender: "Tarantool-patches" After this patch, MAP values can be used as bind variables. However, due to the current syntax for binding in Lua, the only possible way is to use MAP values as the named bind variable. Part of #4763 --- https://github.com/tarantool/tarantool/issues/4763 https://github.com/tarantool/tarantool/tree/imeevma/gh-4763-bindings-for-map src/box/bind.c | 7 +++---- src/box/lua/execute.c | 5 +---- src/box/sql/sqlInt.h | 3 +++ src/box/sql/vdbeapi.c | 8 ++++++++ test/sql-tap/map.test.lua | 25 ++++++++++++++++++++++++- test/sql/bind.result | 6 +++++- 6 files changed, 44 insertions(+), 10 deletions(-) diff --git a/src/box/bind.c b/src/box/bind.c index af9f9eac5..441c9f46f 100644 --- a/src/box/bind.c +++ b/src/box/bind.c @@ -100,15 +100,12 @@ sql_bind_decode(struct sql_bind *bind, int i, const char **packet) bind->s = mp_decode_bin(packet, &bind->bytes); break; case MP_ARRAY: + case MP_MAP: case MP_EXT: bind->s = *packet; mp_next(packet); bind->bytes = *packet - bind->s; break; - case MP_MAP: - diag_set(ClientError, ER_SQL_BIND_TYPE, "MAP", - sql_bind_name(bind)); - return -1; default: unreachable(); } @@ -189,6 +186,8 @@ sql_bind_column(struct sql_stmt *stmt, const struct sql_bind *p, return sql_bind_bin_static(stmt, pos, p->s, p->bytes); case MP_ARRAY: return sql_bind_array_static(stmt, pos, p->s, p->bytes); + case MP_MAP: + return sql_bind_map_static(stmt, pos, p->s, p->bytes); case MP_EXT: assert(p->ext_type == MP_UUID || p->ext_type == MP_DECIMAL); if (p->ext_type == MP_UUID) diff --git a/src/box/lua/execute.c b/src/box/lua/execute.c index 71d4d7fae..b3f81d709 100644 --- a/src/box/lua/execute.c +++ b/src/box/lua/execute.c @@ -386,6 +386,7 @@ lua_sql_bind_decode(struct lua_State *L, struct sql_bind *bind, int idx, int i) diag_set(ClientError, ER_SQL_BIND_TYPE, "USERDATA", sql_bind_name(bind)); return -1; + case MP_MAP: case MP_ARRAY: { size_t used = region_used(region); struct mpstream stream; @@ -410,10 +411,6 @@ lua_sql_bind_decode(struct lua_State *L, struct sql_bind *bind, int idx, int i) diag_set(OutOfMemory, bind->bytes, "region_join", "bind->s"); return -1; } - case MP_MAP: - diag_set(ClientError, ER_SQL_BIND_TYPE, "MAP", - sql_bind_name(bind)); - return -1; default: unreachable(); } diff --git a/src/box/sql/sqlInt.h b/src/box/sql/sqlInt.h index 716110edc..0db16b293 100644 --- a/src/box/sql/sqlInt.h +++ b/src/box/sql/sqlInt.h @@ -559,6 +559,9 @@ sql_bind_bin_static(sql_stmt *stmt, int i, const char *str, uint32_t size); int sql_bind_array_static(sql_stmt *stmt, int i, const char *str, uint32_t size); +int +sql_bind_map_static(sql_stmt *stmt, int i, const char *str, uint32_t size); + int sql_bind_uuid(struct sql_stmt *stmt, int i, const struct tt_uuid *uuid); diff --git a/src/box/sql/vdbeapi.c b/src/box/sql/vdbeapi.c index 3ea155d17..060846572 100644 --- a/src/box/sql/vdbeapi.c +++ b/src/box/sql/vdbeapi.c @@ -532,6 +532,14 @@ sql_bind_array_static(sql_stmt *stmt, int i, const char *str, uint32_t size) return sql_bind_type(vdbe, i, "array"); } +int +sql_bind_map_static(sql_stmt *stmt, int i, const char *str, uint32_t size) +{ + struct Vdbe *vdbe = (struct Vdbe *)stmt; + mem_set_map_static(&vdbe->aVar[i - 1], (char *)str, size); + return sql_bind_type(vdbe, i, "map"); +} + int sql_bind_uuid(struct sql_stmt *stmt, int i, const struct tt_uuid *uuid) { diff --git a/test/sql-tap/map.test.lua b/test/sql-tap/map.test.lua index 1afbb2b1d..8cdbb1662 100755 --- a/test/sql-tap/map.test.lua +++ b/test/sql-tap/map.test.lua @@ -1,6 +1,6 @@ #!/usr/bin/env tarantool local test = require("sqltester") -test:plan(110) +test:plan(112) box.schema.func.create('M1', { language = 'Lua', @@ -982,6 +982,29 @@ test:do_catchsql_test( 1, "Failed to execute SQL statement: wrong arguments for function ZEROBLOB()" }) +-- Make sure that MAP values can be used as a bound variable. +test:do_test( + "builtins-13.1", + function() + local res = box.execute([[SELECT #a;]], {{['#a'] = {abc = 2, [1] = 3}}}) + return {res.rows[1][1]} + end, { + {abc = 2, [1] = 3} + }) + +local remote = require('net.box') +box.cfg{listen = os.getenv('LISTEN')} +local cn = remote.connect(box.cfg.listen) +test:do_test( + "builtins-13.2", + function() + local res = cn:execute([[SELECT #a;]], {{['#a'] = {abc = 2, [1] = 3}}}) + return {res.rows[1][1]} + end, { + {abc = 2, [1] = 3} + }) +cn:close() + box.execute([[DROP TABLE t1;]]) box.execute([[DROP TABLE t;]]) diff --git a/test/sql/bind.result b/test/sql/bind.result index f269056e2..e738a91cd 100644 --- a/test/sql/bind.result +++ b/test/sql/bind.result @@ -276,7 +276,11 @@ parameters[1][':value'] = {kek = 300} ... execute('SELECT :value', parameters) --- -- error: Bind value type MAP for parameter ':value' is not supported +- metadata: + - name: COLUMN_1 + type: map + rows: + - [{'kek': 300}] ... -- gh-3810: bind values of integer in range up to 2^64 - 1. -- -- 2.25.1