From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <vdavydov.dev@gmail.com>
Date: Mon, 15 Oct 2018 20:29:17 +0300
From: Vladimir Davydov <vdavydov.dev@gmail.com>
Subject: Re: [PATCH v4 02/14] box: introduce key_def_parts_are_sequential
Message-ID: <20181015172917.ho7vsaij6mo5tebl@esperanza>
References: <cover.1539244271.git.kshcherbatov@tarantool.org>
 <b96d6944db09b666dd5d17db386b2da48b811260.1539244271.git.kshcherbatov@tarantool.org>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <b96d6944db09b666dd5d17db386b2da48b811260.1539244271.git.kshcherbatov@tarantool.org>
To: Kirill Shcherbatov <kshcherbatov@tarantool.org>
Cc: tarantool-patches@freelists.org
List-ID: <tarantool-patches.dev.tarantool.org>

On Thu, Oct 11, 2018 at 10:58:47AM +0300, Kirill Shcherbatov wrote:
> Introduced a new key_def_parts_are_sequential routine that test,
> does specified key_def have sequential fields. This would be
> useful with introducing JSON path as there would be another
> complex criteria as fields with JSONs can't be 'sequential' in
> this meaning.
> 
> Part of #1012.

Again, should be

Needed for #1012

Other than that, the patch looks OK to me.