From d1e2b55c72f543300a1848f6c68fa4096846ed86 Mon Sep 17 00:00:00 2001 From: Steve Yen Date: Wed, 21 Mar 2018 17:47:56 -0700 Subject: [PATCH] scorch zap postingsItr.nextDocNum() maintains allNChunk correctly When PostingsIterator.nextDocNum() moves the 'all' roaring bitmap iterator forwards, it was incorrectly not keeping the allNChunk value aligned. --- index/scorch/segment/zap/posting.go | 1 + 1 file changed, 1 insertion(+) diff --git a/index/scorch/segment/zap/posting.go b/index/scorch/segment/zap/posting.go index 081ec5f6..4b4b4933 100644 --- a/index/scorch/segment/zap/posting.go +++ b/index/scorch/segment/zap/posting.go @@ -620,6 +620,7 @@ func (i *PostingsIterator) nextDocNum() (uint64, bool, error) { } allN = i.all.Next() + allNChunk = allN / i.postings.sb.chunkFactor } if i.currChunk != nChunk || i.currChunkFreqNorm == nil {