From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <korablev@tarantool.org>
Received: from smtp45.i.mail.ru (smtp45.i.mail.ru [94.100.177.105])
 (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 D4A8B469719
 for <tarantool-patches@dev.tarantool.org>;
 Thu,  5 Mar 2020 00:18:42 +0300 (MSK)
Date: Wed, 4 Mar 2020 21:18:41 +0000
From: Nikita Pettik <korablev@tarantool.org>
Message-ID: <20200304211841.GB9655@tarantool.org>
References: <20200210074712.86876-1-arkholga@tarantool.org>
 <20200210111826.GD1110@tarantool.org>
 <2cbb1677-039b-9f91-cde7-9c06821653eb@tarantool.org>
 <20200211124245.GB5168@tarantool.org>
 <20200218124612.GA67746@tarantool.org>
 <714780e3-1383-62da-94f9-0e90542d47aa@tarantool.org>
 <20200219162843.GA68447@tarantool.org>
 <78db8579-e246-385b-6541-5cc70395be97@tarantool.org>
MIME-Version: 1.0
Content-Type: text/plain; charset="utf-8"
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
In-Reply-To: <78db8579-e246-385b-6541-5cc70395be97@tarantool.org>
Subject: Re: [Tarantool-patches] [PATCH v6] memtx: fix out of memory
	handling for rtree
List-Id: Tarantool development patches <tarantool-patches.dev.tarantool.org>
List-Unsubscribe: <https://lists.tarantool.org/mailman/options/tarantool-patches>, 
 <mailto:tarantool-patches-request@dev.tarantool.org?subject=unsubscribe>
List-Archive: <https://lists.tarantool.org/pipermail/tarantool-patches/>
List-Post: <mailto:tarantool-patches@dev.tarantool.org>
List-Help: <mailto:tarantool-patches-request@dev.tarantool.org?subject=help>
List-Subscribe: <https://lists.tarantool.org/mailman/listinfo/tarantool-patches>, 
 <mailto:tarantool-patches-request@dev.tarantool.org?subject=subscribe>
To: Olga Arkhangelskaia <arkholga@tarantool.org>
Cc: tarantool-patches@dev.tarantool.org

On 03 Mar 14:50, Olga Arkhangelskaia wrote:
> Hi Sergey! I need time to check everything!
> 
> I can not use error injection that exists in memtx_index_extent_reserve(),
> 
> because it is used  in memtx_space_replace_all_keys ant it is called prior
> memtx_rtree_index_reserve() .
> 
> If we use memtx_index_extent_reserve(), injection we will not reach the
> place where error with rtree occures.
> 
> So to make sure that we fixed the problem we need
> memtx_rtree_index_reserve() to fail.
> 

Since now patch received two acks, it is ready to be pushed.
Olga, could you please rebase branch on the top of master (there's several
merge conflicts) and provide brief @changelog (a few sentences for casual
Tarantool users concerning visible changes this patch introduces)? As soon
as it is ready, I'll push it. Thanks.