KnowStreaming
KnowStreaming copied to clipboard
[Feature] #1121 support clearInactiveClusterPhyBrokers
请不要在没有先创建Issue的情况下创建Pull Request。
变更的目的是什么
Support #1121, clear specific inactive cluster brokers from mysql table. 支持 #1121,清理掉mysql表中特定集群已停机的brokers。
简短的更新日志
在broker 列表页,增加清扫按钮,实现清理集群已停机的brokers.
验证这一变化
请遵循此清单,以帮助我们快速轻松地整合您的贡献:
- [x] 一个 PR(Pull Request的简写)只解决一个问题,禁止一个 PR 解决多个问题;
- [x] 确保 PR 有对应的 Issue(通常在您开始处理之前创建),除非是书写错误之类的琐碎更改不需要 Issue ;
- [x] 格式化 PR 及 Commit-Log 的标题及内容,例如 #861 。PS:Commit-Log 需要在 Git Commit 代码时进行填写,在 GitHub 上修改不了;
- [x] 编写足够详细的 PR 描述,以了解 PR 的作用、方式和原因;
- [x] 编写必要的单元测试来验证您的逻辑更正。如果提交了新功能或重大更改,请记住在 test 模块中添加 integration-test;
- [x] 确保编译通过,集成测试通过;
当前的实现有几个问题:
- 只删除了ks_km_broker表中status = 0的记录,没有同步清理其他表中的记录(比如ks_km_broker_config), 这里是否要同步处理?
- 不知道前端的icon怎么添加,原本想使用的是这个icon: icon-clearicon. 最后照猫画虎弄了一个别的。图表类型、大小、位置可以请会前端的朋友帮忙改正。