remove field stringtype length limit

Signed-off-by: Xianhui.Lin <[email protected]> modify the message hint Signed-off-by: Xianhui.Lin <[email protected]>
milvus-io · Dec 19, 2024 · a084eae · a084eae
1 parent 306e5e6
commit a084eae
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/internal/proxy/task.go b/internal/proxy/task.go
@@ -1246,14 +1246,11 @@ func (t *alterCollectionFieldTask) PreExecute(ctx context.Context) error {
 			if !IsStringType {
 				return merr.WrapErrParameterInvalid(fieldName, "%s can not modify the maxlength for non-string types", schemapb.DataType_name[dataType])
 			}
-			value, err := strconv.Atoi(prop.Value)
+			_, err := strconv.Atoi(prop.Value)
 			if err != nil {
 				return merr.WrapErrParameterInvalid("%s should be an integer, but got %T", prop.Key, prop.Value)
 			}
 
-			if value > defaultMaxVarCharLength {
-				return merr.WrapErrParameterInvalid("%s exceeds the maximum allowed value 65535", prop.Value)
-			}
 		}
 	}
 

diff --git a/internal/proxy/util.go b/internal/proxy/util.go
@@ -364,8 +364,8 @@ func validateMaxLengthPerRow(collectionName string, field *schemapb.FieldSchema)
 		if err != nil {
 			return err
 		}
-		if maxLengthPerRow > defaultMaxVarCharLength || maxLengthPerRow <= 0 {
-			return merr.WrapErrParameterInvalidMsg("the maximum length specified for a VarChar should be in (0, 65535]")
+		if maxLengthPerRow <= 0 {
+			return merr.WrapErrParameterInvalidMsg("the maximum length specified for a VarChar should be greater than 0")
 		}
 		exist = true
 	}