Skip to content

Commit

Permalink
fixed FT CBO vs ColumnarScan
Browse files Browse the repository at this point in the history
  • Loading branch information
glookka authored and sanikolaev committed Aug 21, 2023
1 parent 2868333 commit 1cc04be
Showing 1 changed file with 2 additions and 4 deletions.
6 changes: 2 additions & 4 deletions src/secondaryindex.cpp
Expand Up @@ -744,6 +744,8 @@ static void MarkAvailableAnalyzers ( CSphVector<SecondaryIndexInfo_t> & dSIInfo,
// this belongs in the CBO, but for now let's just remove the option to evaluate FILTER if ANALYZER is present
// as ANALYZERs are always faster than FILTERs
dSIInfo[i].m_dCapabilities.RemoveValue ( SecondaryIndexType_e::FILTER );
if ( dSIInfo[i].m_eType==SecondaryIndexType_e::FILTER )
dSIInfo[i].m_eType = SecondaryIndexType_e::ANALYZER;
}
}

Expand Down Expand Up @@ -1003,10 +1005,6 @@ CSphVector<SecondaryIndexInfo_t> SelectIterators ( const SelectIteratorCtx_t & t
dCapabilities.ZeroVec();
dBest.ZeroVec();

// if we don't have any options, no need to do cost calculation
if ( dSIInfo.all_of ( []( const auto & tSIInfo ){ return tSIInfo.m_dCapabilities.GetLength()==1; } ) )
return dSIInfo;

const int MAX_TRIES = 1024;
for ( int iTry = 0; iTry < MAX_TRIES; iTry++ )
{
Expand Down

0 comments on commit 1cc04be

Please sign in to comment.