[tarantool-patches] Re: [PATCH v9] sql: add index_def to struct Index

Kirill Yukhin kyukhin at tarantool.org
Wed Jul 4 13:46:11 MSK 2018


Hello,
On 03 июл 14:37, Ivan Koptelov wrote:
> In previous patch versions I didn't fix one test, which prints all error
> codes.
> At failed, because I added new error code. Now the test is fixed.
> > > Now every sqlite struct Index is created with tnt struct
> > > index_def inside. This allows us to use tnt index_def
> > > in work with sqlite indexes in the same manner as with
> > > tnt index and is a step to remove sqlite Index with
> > > tnt index.
> > > Fields coll_array, coll_id_array, aiColumn, sort_order
> > > and zName are removed from Index. All usages of this
> > > fields changed to usage of corresponding index_def
> > > fields.
> > > index_is_unique(), sql_index_collation() and
> > > index_column_count() are removed with calls of
> > > index_def corresponding fields.
> > > 
> > > Closes: #3369
> > > 
> > > ---
> > > Branch:
> > > https://github.com/tarantool/tarantool/tree/sb/gh-3369-use-index-def-in-select-and-where
> > > 
> > > Issue:https://github.com/tarantool/tarantool/issues/3369
I've checked the patch into 2.0 branch.

--
Regards, Kirill Yukhin




More information about the Tarantool-patches mailing list