review comments

section_faiss_vector_index.go

@@ -36,7 +36,7 @@ const defaultFaissOMPThreads = 1
 func init() {
 	rand.Seed(time.Now().UTC().UnixNano())
 	registerSegmentSection(SectionFaissVectorIndex, &faissVectorIndexSection{})
-	invertedIndexExclusionChecks = append(invertedIndexExclusionChecks, func(field index.Field) bool {
+	invertedTextIndexSectionExclusionChecks = append(invertedTextIndexSectionExclusionChecks, func(field index.Field) bool {
 		_, ok := field.(index.VectorField)
 		return ok
 	})

section_inverted_text_index.go

@@ -34,14 +34,14 @@ func init() {
 type invertedTextIndexSection struct {
 }
 
-// This function checks whether the inverted index section should process
-// a particular field, avoiding unnecessary work if another section will handle it.
-var isFieldExcludedFromInvertedIndex = func(field index.Field) bool {
-	for _, excludeField := range invertedIndexExclusionChecks {
+// This function checks whether the inverted text index section should avoid processing
+// a particular field, preventing unnecessary work if another section will handle it.
+var isFieldExcludedFromInvertedTextIndexSection = func(field index.Field) bool {
+	for _, excludeField := range invertedTextIndexSectionExclusionChecks {
 		if excludeField(field) {
 			// atleast one section has agreed to exclude this field
-			// from inverted index processing and has agreed to process it
-			// independently
+			// from inverted text index section processing and has
+			// agreed to process it independently
 			return true
 		}
 	}
@@ -50,11 +50,11 @@ var isFieldExcludedFromInvertedIndex = func(field index.Field) bool {
 	return false
 }
 
-// List of checks to determine if a field is excluded from the inverted index section
-var invertedIndexExclusionChecks = make([]func(field index.Field) bool, 0)
+// List of checks to determine if a field is excluded from the inverted text index section
+var invertedTextIndexSectionExclusionChecks = make([]func(field index.Field) bool, 0)
 
 func (i *invertedTextIndexSection) Process(opaque map[int]resetable, docNum uint32, field index.Field, fieldID uint16) {
-	if !isFieldExcludedFromInvertedIndex(field) {
+	if !isFieldExcludedFromInvertedTextIndexSection(field) {
 		invIndexOpaque := i.getInvertedIndexOpaque(opaque)
 		invIndexOpaque.process(field, fieldID, docNum)
 	}

section_synonym.go

@@ -31,7 +31,7 @@ import (
 
 func init() {
 	registerSegmentSection(SectionSynonymIndex, &synonymIndexSection{})
-	invertedIndexExclusionChecks = append(invertedIndexExclusionChecks, func(field index.Field) bool {
+	invertedTextIndexSectionExclusionChecks = append(invertedTextIndexSectionExclusionChecks, func(field index.Field) bool {
 		_, ok := field.(index.SynonymField)
 		return ok
 	})