From b7cfef81c9a86d590b2a9ccdaed184c3e1bbcdef Mon Sep 17 00:00:00 2001 From: Steve Yen Date: Sat, 3 Mar 2018 11:43:22 -0800 Subject: [PATCH] scorch optimize mem processDocument() dict access This change moves the dict lookup to outside of the loop. --- index/scorch/segment/mem/build.go | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/index/scorch/segment/mem/build.go b/index/scorch/segment/mem/build.go index 57d60dc8..2a2683dc 100644 --- a/index/scorch/segment/mem/build.go +++ b/index/scorch/segment/mem/build.go @@ -252,8 +252,9 @@ func (s *Segment) processDocument(result *index.AnalysisResult) { // now that its been rolled up into docMap, walk that for fieldID, tokenFrequencies := range docMap { + dict := s.Dicts[fieldID] for term, tokenFreq := range tokenFrequencies { - pid := s.Dicts[fieldID][term] - 1 + pid := dict[term] - 1 bs := s.Postings[pid] bs.AddInt(int(docNum)) s.Freqs[pid] = append(s.Freqs[pid], uint64(tokenFreq.Frequency()))