-
-
Notifications
You must be signed in to change notification settings - Fork 557
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
建议增加$$inception_backup_information$$ 表的 host 列长度问题 #421
Comments
该BUG在几年前已在 |
嗷嗷,他的意思是想把 goInception/session/session_backup.go Lines 255 to 280 in e2604a3
这里的buf.WriteString("host VARCHAR(64),") 改大了 |
由于此问题没有最近的活动,因此已被自动标记为陈旧。如果没有进一步的活动,会作为不活跃issue关闭。感谢你对本项目的贡献。 This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
我理解错了,之前修复的是生成备份库名称逻辑。 |
@hanchuanchuan |
是的, 很报歉我之前的修复丢失了, 可能本地误回滚导致的, 所以又重新改了一次. |
请升级到 |
描述您想要的功能:
在执行工单时,我们有时会遇到这个抱错,
【Data too long for column 'host' at row 1.】
排查发现是备份库下的$$inception_backup_information$$ 表的 host 列长度问题,
host 字段的长度是varchar(64) ,我们使用的AWS数据库,有时候连接地址会超过64个字符,
当然,解决这个问题,也很简单,只修改alter table$$inception_backup_information$$ modify host varchar(128) ,加大长度就可以了,
但这并不能从解本上解决,后面添加的新实例,再次执行工单,还是可能会遇到报错问题,虽然这个报错并不影响工单正常执行。
描述您考虑过的解决/替代方案:
我建议修改$$inception_backup_information$$ 建表语句,将 host varchar(64) 修改为 varchar(128)
在 inception 项目 /sql/sql_parse.cc 文件中
链接:https://github.com/hhyo/inception/blob/master/sql/sql_parse.cc
第7994行。
可参考文档,资料,引用等:
The text was updated successfully, but these errors were encountered: