KnowStreaming icon indicating copy to clipboard operation
KnowStreaming copied to clipboard

[Feature] #1121 support clearInactiveClusterPhyBrokers

Open chang-wd opened this issue 1 year ago • 1 comments

请不要在没有先创建Issue的情况下创建Pull Request。

变更的目的是什么

Support #1121, clear specific inactive cluster brokers from mysql table. 支持 #1121,清理掉mysql表中特定集群已停机的brokers。

简短的更新日志

在broker 列表页,增加清扫按钮,实现清理集群已停机的brokers.

验证这一变化

image

请遵循此清单,以帮助我们快速轻松地整合您的贡献:

  • [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] 确保编译通过,集成测试通过;

chang-wd avatar Nov 20 '24 13:11 chang-wd

当前的实现有几个问题:

  1. 只删除了ks_km_broker表中status = 0的记录,没有同步清理其他表中的记录(比如ks_km_broker_config), 这里是否要同步处理?
  2. 不知道前端的icon怎么添加,原本想使用的是这个icon: icon-clearicon. 最后照猫画虎弄了一个别的。图表类型、大小、位置可以请会前端的朋友帮忙改正。

chang-wd avatar Nov 20 '24 14:11 chang-wd