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 D2EA52CC31 for ; Tue, 27 Mar 2018 07:32:00 -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 QzxDHEdmwbnr for ; Tue, 27 Mar 2018 07:32:00 -0400 (EDT) Received: from smtp16.mail.ru (smtp16.mail.ru [94.100.176.153]) (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 3CEFC2CC2D for ; Tue, 27 Mar 2018 07:32:00 -0400 (EDT) Content-Type: text/plain; charset=utf-8 Mime-Version: 1.0 (Mac OS X Mail 11.2 \(3445.5.20\)) Subject: [tarantool-patches] Re: [PATCH 0/4] Cleanup SQL memory and threading subsystems From: "v.shpilevoy@tarantool.org" In-Reply-To: Date: Tue, 27 Mar 2018 14:31:57 +0300 Content-Transfer-Encoding: quoted-printable Message-Id: <5FC64037-E245-4CDC-9985-F9BEEC202227@tarantool.org> References: 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 Cc: Kirill Yukhin LGTM. > 26 =D0=BC=D0=B0=D1=80=D1=82=D0=B0 2018 =D0=B3., =D0=B2 18:56, Kirill = Yukhin =D0=BD=D0=B0=D0=BF=D0=B8=D1=81=D0=B0=D0=BB(= =D0=B0): >=20 > This patcheset perform a cleanup of SQL frontend by removing > dead memory allocators and their APIs and by getting rid of > all mutex-related code. >=20 > Issue: https://github.com/tarantool/tarantool/issues/3192 > Branch: = https://github.com/tarantool/tarantool/tree/kyukhin/gh-3192-remove-sqlite3= >=20 >=20 > Kirill Yukhin (4): > sql: Remove all references to sqlite3.h > sql: remove mutexes around the code > sql: remove dead memory allocation systems > sql: Remove dead defines from sqliteInt.h >=20 > src/box/execute.c | 2 +- > src/box/lua/sql.c | 2 +- > src/box/sql.c | 2 - > src/box/sql/CMakeLists.txt | 9 - > src/box/sql/date.c | 7 - > src/box/sql/func.c | 2 - > src/box/sql/global.c | 8 - > src/box/sql/legacy.c | 2 - > src/box/sql/main.c | 337 +-- > src/box/sql/malloc.c | 195 +- > src/box/sql/mem0.c | 116 - > src/box/sql/mem1.c | 314 --- > src/box/sql/mem2.c | 602 ---- > src/box/sql/mem3.c | 767 ----- > src/box/sql/mem5.c | 648 ----- > src/box/sql/mutex.c | 211 -- > src/box/sql/mutex.h | 82 - > src/box/sql/mutex_noop.c | 300 -- > src/box/sql/mutex_unix.c | 442 --- > src/box/sql/os.c | 23 +- > src/box/sql/os_unix.c | 180 +- > src/box/sql/parse.c | 143 +- > src/box/sql/prepare.c | 8 - > src/box/sql/printf.c | 6 - > src/box/sql/random.c | 11 - > src/box/sql/sqlite3.h | 6728 = -------------------------------------------- > src/box/sql/sqliteInt.h | 778 ++++- > src/box/sql/status.c | 12 +- > src/box/sql/tokenize.c | 2 - > src/box/sql/vdbe.c | 5 +- > src/box/sql/vdbeapi.c | 54 - > src/box/sql/vdbeaux.c | 2 - > src/box/sql/vdbemem.c | 17 - > src/box/sql/vdbesort.c | 6 +- > 34 files changed, 887 insertions(+), 11136 deletions(-) > delete mode 100644 src/box/sql/mem0.c > delete mode 100644 src/box/sql/mem1.c > delete mode 100644 src/box/sql/mem2.c > delete mode 100644 src/box/sql/mem3.c > delete mode 100644 src/box/sql/mem5.c > delete mode 100644 src/box/sql/mutex.c > delete mode 100644 src/box/sql/mutex.h > delete mode 100644 src/box/sql/mutex_noop.c > delete mode 100644 src/box/sql/mutex_unix.c > delete mode 100644 src/box/sql/sqlite3.h >=20 > --=20 > 2.11.0 >=20 >=20