gibheer
/
bleve
0
0
Fork 0
Code

scorch simplify err check after vellum load

This commit is contained in:
Steve Yen 2017-12-19 22:26:17 -08:00
parent 4d28b16896
commit 1abbfadf0d
1 changed files with 3 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
index/scorch/segment/zap/segment.go
View File

@ -159,8 +159,8 @@ func (s *Segment) Dictionary(field string) (segment.TermDictionary, error) {
return dict, err
}
func (s *Segment) dictionary(field string) (*Dictionary, error) {
rv := &Dictionary{
func (s *Segment) dictionary(field string) (rv *Dictionary, err error) {
rv = &Dictionary{
segment: s,
field: field,
}
@ -170,19 +170,15 @@ func (s *Segment) dictionary(field string) (*Dictionary, error) {
rv.fieldID = rv.fieldID - 1
dictStart := s.fieldsOffsets[rv.fieldID]
if dictStart > 0 {
// read the length of the vellum data
vellumLen, read := binary.Uvarint(s.mm[dictStart : dictStart+binary.MaxVarintLen64])
fstBytes := s.mm[dictStart+uint64(read) : dictStart+uint64(read)+vellumLen]
if fstBytes != nil {
fst, err := vellum.Load(fstBytes)
rv.fst, err = vellum.Load(fstBytes)
if err != nil {
return nil, fmt.Errorf("dictionary field %s vellum err: %v", field, err)
}
if err == nil {
rv.fst = fst
}
}
}