scriptoriumadmin/featurebase
1
0
Fork 0
mirror of https://github.com/featurebasedb/featurebase.git synced 2026-09-07 17:15:56 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Merge pull request #846 from travisturner/reduce-executor-locks

Browse source 
micro optimization: reduce Lock calls in executor
This commit is contained in:
Travis Turner 2017-09-25 17:12:17 -05:00 • committed by GitHub
commit 046fb9b97e
1 changed files with 6 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
executor.go
View file

@ -92,8 +92,12 @@ func (e *Executor) Execute(ctx context.Context, index string, q *pql.Query, slic
// Determine slices and inverseSlices for use in e.executeCall().
if needsSlices {
// Round up the number of slices.
maxSlice := e.Holder.Index(index).MaxSlice()
maxInverseSlice := e.Holder.Index(index).MaxInverseSlice()
idx := e.Holder.Index(index)
if idx == nil {
return nil, ErrIndexNotFound
}
maxSlice := idx.MaxSlice()
maxInverseSlice := idx.MaxInverseSlice()
// Generate a slices of all slices.
slices = make([]uint64, maxSlice+1)
@ -108,10 +112,6 @@ func (e *Executor) Execute(ctx context.Context, index string, q *pql.Query, slic
}
// Fetch column label from index.
idx := e.Holder.Index(index)
if idx == nil {
return nil, ErrIndexNotFound
}
columnLabel = idx.ColumnLabel()
}
}