apolloconfig / apollo

Apollo is a reliable configuration management system suitable for microservice configuration management scenarios.
https://www.apolloconfig.com
Apache License 2.0
29.2k stars 10.21k forks source link

Apollo不同表的的相同业务含义的字段长度不一致 #5259

Closed youngzil closed 3 weeks ago

youngzil commented 1 month ago

你的特性请求和某个问题有关吗?请描述

在实际使用时,发现日志经常会有少量的数据库字段超长的报错,发现是Release表的NamespaceName字段是 varchar(500),但是InstanceConfig表的ConfigNamespaceName字段是 varchar(32)

  1. 比如NamespaceName,在AppNamespace表中Name 字段是varchar(32)
  2. 比如Cluster字段,Cluster表中Name字段是 varchar(32),Commit表中ClusterName字段是 varchar(500)

相同含义的字段在所有表中是否有必要同一,避免一些潜在的问题

清晰简洁地描述一下你希望的解决方案

清晰简洁地描述一下这个特性的备选方案

其它背景

在这里添加和这个特性请求有关的背景说明、截图

youngzil commented 1 month ago

I saw that there was a PR #4725 that made some optimizations for the field length unification, but it did not include the three tables commit, namespace, and release. Why is this?

nobodyiam commented 1 month ago

It includes the commit, namespace, and release tables but appears to miss the NamespaceName and ClusterName fields.

youngzil commented 1 month ago

It includes the commit, namespace, and release tables but appears to miss the NamespaceName and ClusterName fields.

I think this can be fixed and is safe.