From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from localhost (localhost [127.0.0.1]) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTP id ADB8E30454 for ; Tue, 11 Jun 2019 06:00:11 -0400 (EDT) Received: from turing.freelists.org ([127.0.0.1]) by localhost (turing.freelists.org [127.0.0.1]) (amavisd-new, port 10024) with ESMTP id jzjtf_O_DZAF for ; Tue, 11 Jun 2019 06:00:11 -0400 (EDT) Received: from smtpng1.m.smailru.net (smtpng1.m.smailru.net [94.100.181.251]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by turing.freelists.org (Avenir Technologies Mail Multiplex) with ESMTPS id E29E330450 for ; Tue, 11 Jun 2019 06:00:10 -0400 (EDT) Received: by smtpng1.m.smailru.net with esmtpa (envelope-from ) id 1hadZY-0002T4-JH for tarantool-patches@freelists.org; Tue, 11 Jun 2019 13:00:09 +0300 Subject: [tarantool-patches] Re: [PATCH v1 00/28] sql: Remove SQL error system References: From: Imeev Mergen Message-ID: <2ddee036-406d-a8b7-2986-585d225f7195@tarantool.org> Date: Tue, 11 Jun 2019 13:00:08 +0300 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit Content-Language: en-US Sender: tarantool-patches-bounce@freelists.org Errors-to: tarantool-patches-bounce@freelists.org Reply-To: tarantool-patches@freelists.org List-Help: List-Unsubscribe: List-software: Ecartis version 1.0.0 List-Id: tarantool-patches List-Subscribe: List-Owner: List-post: List-Archive: To: tarantool-patches@freelists.org Sorry, I did not include the links in the original letter: https://github.com/tarantool/tarantool/issues/4074 https://github.com/tarantool/tarantool/tree/imeevma/gh-4074-follow-ups On 6/10/19 4:56 PM, imeevma@tarantool.org wrote: > This patch-set removes unused and unnecessary code of SQL error > system. > > Mergen Imeev (26): > sql: remove field zErrMsg from struct Vdbe > sql: remove field pErr from struct sql > sql: remove field errCode from struct sql > sql: remove sqlError() and remove sqlErrorWithMsg() > sql: remove unused functions of SQL error system > sql: disable lookaside system > sql: remove SQL_OK error/status code > sql: remove SQL_PERM, SQL_WARNING, SQL_ABORT errcodes > sql: remove SQL_CANTOPEN errcode > sql: remove SQL_NOTFOUND error/status code > sql: remove SQL_LOCKED errcode > sql: remove SQL_FULL errcode > sql: remove SQL_MISUSE errcode > sql: remove SQL_RANGE errcode > sql: remove SQL_SCHEMA errcode > sql: remove SQL_TOOBIG errcode > sql: remove SQL_BUSY errcode > sql: remove SQL_CONSTRAINT errcode > sql: remove SQL_ERROR errcode > sql: remove SQL_NOMEM errcode > sql: remove SQL_IOERR errcode > sql: remove SQL_TARANTOOL_ERROR errcode > sql: remove field errMask from struct sql > sql: replace rc by is_aborted in struct VDBE > sql: remove sql_log() > sql: remove function sqlApiExit() > > Nikita Pettik (2): > sql: cleanup of legacy memory management system > sql: make function return void instead of int > > src/box/bind.c | 37 +--- > src/box/ck_constraint.c | 4 +- > src/box/execute.c | 6 +- > src/box/lua/lua_sql.c | 2 +- > src/box/sql.c | 104 ++++------ > src/box/sql/analyze.c | 2 +- > src/box/sql/build.c | 6 +- > src/box/sql/cursor.c | 4 +- > src/box/sql/date.c | 14 +- > src/box/sql/expr.c | 7 +- > src/box/sql/fault.c | 4 +- > src/box/sql/fk_constraint.c | 8 +- > src/box/sql/func.c | 16 +- > src/box/sql/global.c | 3 - > src/box/sql/insert.c | 9 +- > src/box/sql/legacy.c | 21 +- > src/box/sql/main.c | 266 ++++--------------------- > src/box/sql/malloc.c | 445 +----------------------------------------- > src/box/sql/os.c | 14 +- > src/box/sql/os_unix.c | 281 ++++++++------------------ > src/box/sql/prepare.c | 73 ++----- > src/box/sql/printf.c | 36 +--- > src/box/sql/resolve.c | 3 +- > src/box/sql/select.c | 61 +++--- > src/box/sql/sqlInt.h | 173 +--------------- > src/box/sql/status.c | 182 ----------------- > src/box/sql/tarantoolInt.h | 11 +- > src/box/sql/tokenize.c | 5 - > src/box/sql/trigger.c | 2 +- > src/box/sql/util.c | 162 --------------- > src/box/sql/vdbe.c | 111 ++++------- > src/box/sql/vdbeInt.h | 13 +- > src/box/sql/vdbeapi.c | 357 +++++++++------------------------ > src/box/sql/vdbeaux.c | 217 ++++++-------------- > src/box/sql/vdbemem.c | 120 ++++++------ > src/box/sql/vdbesort.c | 274 ++++++++++++-------------- > src/box/sql/where.c | 108 +++++----- > src/box/sql/whereexpr.c | 3 +- > test/sql/bind.result | 8 + > test/sql/bind.test.lua | 3 + > test/sql/func-recreate.result | 3 +- > 41 files changed, 726 insertions(+), 2452 deletions(-) >