Skip to content

Conversation

@HZSUZJ
Copy link
Contributor

@HZSUZJ HZSUZJ commented Jul 1, 2023

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

变更的目的是什么

需要允许为不存在的Topic添加ACL权限,可能出现Topic还没有,但是想提前增加权限的动作;

简短的更新日志

允许为不存在的Topic添加ACL权限

验证这一变化

所有的topic

02813ae4e3d95eedc9762f129a03ae6

改变前:

2ac3f514cbdfe1358191e5df00a6c3a

改变后:

174e3b88297aa650d955e83324417fe

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

  • 一个 PR(Pull Request的简写)只解决一个问题,禁止一个 PR 解决多个问题;
  • 确保 PR 有对应的 Issue(通常在您开始处理之前创建),除非是书写错误之类的琐碎更改不需要 Issue ;
  • 格式化 PR 及 Commit-Log 的标题及内容,例如 [Bugfix]修复用户信息修改后,用户列表会抛出空指针异常的问题(#860) #861 。PS:Commit-Log 需要在 Git Commit 代码时进行填写,在 GitHub 上修改不了;
  • 编写足够详细的 PR 描述,以了解 PR 的作用、方式和原因;
  • 编写必要的单元测试来验证您的逻辑更正。如果提交了新功能或重大更改,请记住在 test 模块中添加 integration-test;
  • 确保编译通过,集成测试通过;

@ZQKC
Copy link
Collaborator

ZQKC commented Jul 2, 2023

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

变更的目的是什么

需要允许为不存在的Topic添加ACL权限,可能出现Topic还没有,但是想提前增加权限的动作;

简短的更新日志

允许为不存在的Topic添加ACL权限

验证这一变化

所有的topic

02813ae4e3d95eedc9762f129a03ae6

改变前:

2ac3f514cbdfe1358191e5df00a6c3a

改变后:

174e3b88297aa650d955e83324417fe

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

  • 一个 PR(Pull Request的简写)只解决一个问题,禁止一个 PR 解决多个问题;
  • 确保 PR 有对应的 Issue(通常在您开始处理之前创建),除非是书写错误之类的琐碎更改不需要 Issue ;
  • 格式化 PR 及 Commit-Log 的标题及内容,例如 [Bugfix]修复用户信息修改后,用户列表会抛出空指针异常的问题(#860) #861 。PS:Commit-Log 需要在 Git Commit 代码时进行填写,在 GitHub 上修改不了;
  • 编写足够详细的 PR 描述,以了解 PR 的作用、方式和原因;
  • 编写必要的单元测试来验证您的逻辑更正。如果提交了新功能或重大更改,请记住在 test 模块中添加 integration-test;
  • 确保编译通过,集成测试通过;

1、后续commit-log里面不需要加issue的id,直接就是:[Bugfix]修复消费组Offset重置后,提示重置成功,但是前端不刷新数据,Offset无变化的问题,就可以了;
2、一个pr可以存在多个commit;
3、原先的逻辑不要修改掉,仅仅是在prefixed的这个模式下,不需要校验Topic是否存在;

@HZSUZJ HZSUZJ closed this Jul 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants