gibheer
/
bleve
0
0
Fork 0
Code

scorch zap reuse roaring Bitmap in prepareDicts() slice growth 

In this change, if the postings/postingsLocs slices need to be grown,
then copy over and reuse any of the preallocated roaring Bitmap's from
the old slice.
This commit is contained in:
Steve Yen 2018-03-11 21:07:14 -07:00
parent b1f3969521
commit 07901910e2
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
index/scorch/segment/zap/new.go
View File

@ -309,19 +309,27 @@ func (s *interim) prepareDicts() {
if cap(s.Postings) >= numPostingsLists {
s.Postings = s.Postings[:numPostingsLists]
} else {
s.Postings = make([]*roaring.Bitmap, numPostingsLists)
postings := make([]*roaring.Bitmap, numPostingsLists)
copy(postings, s.Postings[:cap(s.Postings)])
for i := 0; i < numPostingsLists; i++ {
s.Postings[i] = roaring.New()
if postings[i] == nil {
postings[i] = roaring.New()
}
}
s.Postings = postings
}
if cap(s.PostingsLocs) >= numPostingsLists {
s.PostingsLocs = s.PostingsLocs[:numPostingsLists]
} else {
s.PostingsLocs = make([]*roaring.Bitmap, numPostingsLists)
postingsLocs := make([]*roaring.Bitmap, numPostingsLists)
copy(postingsLocs, s.PostingsLocs[:cap(s.PostingsLocs)])
for i := 0; i < numPostingsLists; i++ {
s.PostingsLocs[i] = roaring.New()
if postingsLocs[i] == nil {
postingsLocs[i] = roaring.New()
}
}
s.PostingsLocs = postingsLocs
}
// TODO: reuse this.