[Tarantool-patches] [PATCH] sql: fix index consideration with INDEXED BY clause

Nikita Pettik korablev at tarantool.org
Fri Dec 27 17:20:43 MSK 2019


On 27 Dec 13:48, Nikita Pettik wrote:
> On 27 Dec 14:31, Vladislav Shpilevoy wrote:
> > Hi! Thanks for the patch! LGTM.
> 
> Pushed to master.

I've also backported to 2.2 (since without this fix INDEXED BY is
almost useless).
  
> > On 26/12/2019 01:48, Nikita Pettik wrote:
> > > Accidentally, number of indexes to be considered during query planning
> > > in presence of INDEXED BY is calculated wrong. Instead of one (INDEXED
> > > BY is not a hint but requirement) index to be used (which is indicated
> > > in INDEXED BY clause), all space indexes take part in query planning.
> > > There are not so many tests checking this feature, so unfortunately this
> > > bug was hidden. Let's fix it and force only one index to be used in QP
> > > in case of INDEXED BY clause.
> > > ---
> > > Branch: https://github.com/tarantool/tarantool/tree/np/sql-indexed-by-fix


More information about the Tarantool-patches mailing list