Skip to content

fix: 删除 CLA 更新后的立即通知,统一走定时器扫描#485

Open
JavaPythonAIForBAT wants to merge 3 commits into
reviewfrom
fix/remove-immediate-notify
Open

fix: 删除 CLA 更新后的立即通知,统一走定时器扫描#485
JavaPythonAIForBAT wants to merge 3 commits into
reviewfrom
fix/remove-immediate-notify

Conversation

@JavaPythonAIForBAT

Copy link
Copy Markdown
Contributor

Summary

  • 删除 CLA 更新后的立即通知触发(TriggerNotify)
  • SetPendingCLAForLink 保留,持久化通知标记
  • 通知统一走定时器(12:00 + 0:00)扫描

原因

大社区场景下立即触发受 batch_size=500 限制只能发 500 封,剩余仍需等定时器,立即无实际意义。删除后统一走定时器,逻辑更简单可靠。

大社区场景下立即触发 500 封限制无实际价值,定时器中午/午夜各扫一轮即可。
SetPendingCLAForLink 保留,持久化通知标记,定时器扫到后自动发送。
@opensourceways-bot

Copy link
Copy Markdown

Welcome To opensourceways Community

Hey @JavaPythonAIForBAT , thanks for your contribution to the community.

Bot Usage Manual

I'm the Bot here serving you. You can find the instructions on how to interact with me at Here . That means you can comment below every pull request or issue to trigger Bot Commands.

Contact Guide

If you have any questions, please contact the SIG: infratructure ,
and any of the maintainers: @GeorgeCao-hw, @TangJia025, @pkking, @zhongjun2 ,
and any of the committers: @JavaPythonAIForBAT, @Zherphy, @tfhddd, @zhongjun2 .

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

JavaPythonAIForBAT, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown

Linking Issue Notice

@JavaPythonAIForBAT , the pull request must be linked to at least one issue.
If an issue has already been linked, but the needs-issue label remains, you can remove the label by commenting /check-issue .

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request removes the unused import of the "watch" package and the call to "watch.TriggerNotify()" within the "Update" method of "claService" in "signing/app/cla.go". There are no review comments, so I have no feedback to provide.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread signing/app/cla.go
logs.Error("set pending CLA for link failed: %s, err: %v", cmd.LinkId, err)
}

watch.TriggerNotify()

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

需要把冗余代码一起删掉

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

已经删除了

- 删除 TriggerNotify() 函数
- 删除 corpTrigger/individualTrigger 通道字段及初始化
- 删除 watcher goroutine 中的 trigger select case
- 通知统一走定时器(12:00 + 0:00),不再保留即时触发通道
@opensourceways-bot

Copy link
Copy Markdown
检查项 状态
敏感信息扫描
漏洞扫描
Check代码检查
开源license合规扫描
开发阶段设计文档检查
流水线链接 点击跳转查看日志

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

JavaPythonAIForBAT, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown
检查项 状态
敏感信息扫描
漏洞扫描
Check代码检查
开源license合规扫描
开发阶段设计文档检查
流水线链接 点击跳转查看日志

@opensourceways-bot

Copy link
Copy Markdown

CLA Signature Pass

JavaPythonAIForBAT, thanks for your pull request. All authors of the commits have signed the CLA. 👍

@opensourceways-bot

Copy link
Copy Markdown
检查项 状态
敏感信息扫描
漏洞扫描
Check代码检查
开源license合规扫描
开发阶段设计文档检查
流水线链接 点击跳转查看日志

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants