MB-60569: Set segmentVersion to 16 when index updated to hold vectors

abhinavdangeti · abhinavdangeti · commit c5fd0e998b73 · 2024-01-29T21:40:33.000Z
Change-Id: I58b86d933cbb29911dcf03a993f158b92b1ccade Reviewed-on: https://review.couchbase.org/c/cbft/+/204625 Well-Formed: Restriction Checker Well-Formed: Build Bot <build@couchbase.com> Tested-by: Abhi Dangeti <abhinav@couchbase.com> Reviewed-by: Abhi Dangeti <abhinav@couchbase.com>
diff --git a/pindex_bleve.go b/pindex_bleve.go
@@ -627,8 +627,8 @@ func PrepareIndexDef(mgr *cbgt.Manager, indexDef *cbgt.IndexDef) (
 			}
 
 			if vectorFieldsExistWithinIndexMapping(bp.Mapping) && int(zv) < BleveVectorZapVersion {
-				return nil, cbgt.NewBadRequestError("PrepareIndex, err: zap version %d isn't "+
-					"supported for vectors' datatype and search", int(zv))
+				// overrride segmentVersion to minimum version needed to support vector mappings
+				bp.Store["segmentVersion"] = BleveVectorZapVersion
 			}
 		} else {
 			return nil, cbgt.NewBadRequestError("PrepareIndex, err: segmentVersion %v "+

Original file line number	Diff line number	Diff line change
`@@ -627,8 +627,8 @@ func PrepareIndexDef(mgr cbgt.Manager, indexDef cbgt.IndexDef) (`
`627`	`627`	`}`
`628`	`628`
`629`	`629`	`if vectorFieldsExistWithinIndexMapping(bp.Mapping) && int(zv) < BleveVectorZapVersion {`
`630`		`- return nil, cbgt.NewBadRequestError("PrepareIndex, err: zap version %d isn't "+`
`631`		`- "supported for vectors' datatype and search", int(zv))`
	`630`	`+ // overrride segmentVersion to minimum version needed to support vector mappings`
	`631`	`+ bp.Store["segmentVersion"] = BleveVectorZapVersion`
`632`	`632`	`}`
`633`	`633`	`} else {`
`634`	`634`	`return nil, cbgt.NewBadRequestError("PrepareIndex, err: segmentVersion %v "+`