You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@inlong.apache.org by GitBox <gi...@apache.org> on 2022/10/18 03:33:30 UTC

[GitHub] [inlong] fuweng11 commented on a diff in pull request #6200: [INLONG-6103][Manager] Support update and save cluster switch info

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


##########
inlong-manager/manager-pojo/src/main/java/org/apache/inlong/manager/pojo/group/InlongGroupInfo.java:
##########
@@ -116,6 +116,12 @@ public abstract class InlongGroupInfo {
     @JsonFormat(pattern = "yyyy-MM-dd HH:mm:ss")
     private Date modifyTime;
 
+    @ApiModelProperty(value = "Backup inlong cluster tag")
+    private String backupInlongClusterTag;

Review Comment:
   If this parameter is needed in all group information, why not add a column to store it in the database instead of putting it in extParams?



-- 
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