[tarantool-patches] Re: [PATCH v4 0/2] memtx: add yields during index build

Serge Petrenko sergepetrenko at tarantool.org
Thu May 30 16:41:14 MSK 2019



> 30 мая 2019 г., в 16:38, Vladimir Davydov <vdavydov.dev at gmail.com> написал(а):
> 
> On Thu, May 30, 2019 at 03:13:26PM +0300, Serge Petrenko wrote:
>> This patchset makes memtx engine yield during index build, which prevents it
>> from stalling the event loop, and moves appropriate test cases from vinyl suite,
>> where such a feature was implemented earlier, to engine suite.
>> 
>> The first patch introduces changes in index build mechanism
>> The second patch adds necessary tests
>> 
>> Changes in v4:
>>  - move errinj introduction to the
>>    second patch
>>  - move more tests from vinyl/errinj_ddl
>>    to engine/errinj_ddl
>>  - unify errinjs in memtx and vinyl index
>>    build for easier testing
>>  - ref count cursor before yield in index
>>    build
>> 
>> Changes in v3:
>> - split the patch into two,
>>   the second one moving appropriate
>>   tests from vinyl to engine suite
>> - add a docbot request to the
>>   first patch
>> - ensure index unique constraints
>>   are checked in on_replace triggers
>> - fix tuple comparsion in on_replace
>>   triggers
>> 
>> Changes in v2:
>> - add an on_replace trigger
>>   to handle concurrent replaces
>>   while index build yields
>> - modify test case slightly,
>>   test concurrent replaces.
>> 
>> Serge Petrenko (2):
>>  memtx: add yields during index build
>>  test: move background index build test to engine suite from vinyl
> 
> Pushed to master. Thanks for addressing all my comments.
> 
> We should probably make format check yielding, too. This should be
> very similar to what you've just done. Please consider doing this in
> a follow-up patch.

Ok, will do. Thank you

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.tarantool.org/pipermail/tarantool-patches/attachments/20190530/a8886bfc/attachment.html>


More information about the Tarantool-patches mailing list