gibheer
/
bleve
0
0
Fork 0
Code

Fixing the scorch search request memory estimate 

Do not re-account for certain referenced data in the zap structures.

New estimates:

                                    ESTIMATE    BENCHMEM
TermQuery                           11396       12437
MatchQuery                          12244       12951
DisjunctionQuery (Term queries)     20644       20709
This commit is contained in:
abhinavdangeti 2018-03-06 15:53:14 -08:00
parent ae81806435
commit 5c721226cf
1 changed files with 0 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
index/scorch/segment/zap/posting.go
View File

@ -57,18 +57,6 @@ type PostingsList struct {
func (p *PostingsList) Size() int {
sizeInBytes := reflectStaticSizePostingsList + size.SizeOfPtr
if p.sb != nil {
sizeInBytes += (p.sb.Size() - len(p.sb.mem)) // do not include the mmap'ed part
}
if p.locBitmap != nil {
sizeInBytes += int(p.locBitmap.GetSizeInBytes())
}
if p.postings != nil {
sizeInBytes += int(p.postings.GetSizeInBytes())
}
if p.except != nil {
sizeInBytes += int(p.except.GetSizeInBytes())
}