Skip to content

Commit

Permalink
Fix field indexing
Browse files Browse the repository at this point in the history
  • Loading branch information
xichen2020 committed Jan 28, 2019
1 parent 14b037a commit 1c68e44
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 6 deletions.
5 changes: 3 additions & 2 deletions storage/immutable_segment.go
Original file line number Diff line number Diff line change
Expand Up @@ -218,10 +218,11 @@ func (s *immutableSeg) QueryRaw(
}
}()

if queryFields[rawDocSourceFieldIdx] == nil {
queryRawDocSourceFieldIdx := fieldIndexMap[rawDocSourceFieldIdx]
if queryFields[queryRawDocSourceFieldIdx] == nil {
return errNoRawDocSourceField
}
rawDocSourceField, ok := queryFields[rawDocSourceFieldIdx].StringField()
rawDocSourceField, ok := queryFields[queryRawDocSourceFieldIdx].StringField()
if !ok {
return errNoStringValuesInRawDocSourceField
}
Expand Down
5 changes: 3 additions & 2 deletions storage/mutable_segment.go
Original file line number Diff line number Diff line change
Expand Up @@ -213,10 +213,11 @@ func (s *mutableSeg) QueryRaw(
return nil, nil
}

if queryFields[rawDocSourceFieldIdx] == nil {
queryRawDocSourceFieldIdx := fieldIndexMap[rawDocSourceFieldIdx]
if queryFields[queryRawDocSourceFieldIdx] == nil {
return nil, errNoRawDocSourceField
}
rawDocSourceField, ok := queryFields[rawDocSourceFieldIdx].StringField()
rawDocSourceField, ok := queryFields[queryRawDocSourceFieldIdx].StringField()
if !ok {
return nil, errNoStringValuesInRawDocSourceField
}
Expand Down
4 changes: 2 additions & 2 deletions storage/segment_query.go
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,8 @@ func applyFilters(
queryFields []indexfield.DocsField,
numTotalDocs int32,
) (index.DocIDSetIterator, error) {
timestampFieldIdx := fieldIndexMap[timestampFieldIdx]
timestampField, exists := queryFields[timestampFieldIdx].TimeField()
queryTimestampFieldIdx := fieldIndexMap[timestampFieldIdx]
timestampField, exists := queryFields[queryTimestampFieldIdx].TimeField()
if !exists {
return nil, errNoTimeValuesInTimestampField
}
Expand Down

0 comments on commit 1c68e44

Please sign in to comment.