You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@inlong.apache.org by "fuweng11 (via GitHub)" <gi...@apache.org> on 2023/06/20 03:17:24 UTC

[GitHub] [inlong] fuweng11 commented on a diff in pull request #8283: [INLONG-8282][Manager] Ensure that the field length in the request is consistent with the length limit in DB

fuweng11 commented on code in PR #8283:
URL: https://github.com/apache/inlong/pull/8283#discussion_r1234687231


##########
inlong-manager/manager-pojo/src/main/java/org/apache/inlong/manager/pojo/group/InlongGroupRequest.java:
##########
@@ -49,8 +49,8 @@ public abstract class InlongGroupRequest extends BaseInlongGroup {
 
     @ApiModelProperty(value = "Inlong group id", required = true)
     @NotBlank(message = "inlongGroupId cannot be blank")
-    @Length(min = 4, max = 100, message = "length must be between 4 and 100")
-    @Pattern(regexp = "^[a-z0-9_.-]{4,100}$", message = "only supports lowercase letters, numbers, '-', or '_'")
+    @Length(min = 4, max = 256, message = "length must be between 4 and 256")

Review Comment:
   done



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@inlong.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org