Duansg opened a new pull request, #3914: URL: https://github.com/apache/hertzbeat/pull/3914
## What's changed? close #3903 1. The `org.apache.hertzbeat.collector.timer.TimerDispatcher#goOnline` method will clear all current periodic tasks and temporary tasks. 2. When `ManageServer` receives a `GO_ONLINE` message, it returns an acknowledgment message. If no validation is performed, `Collector` will trigger `goOnline()` again upon receiving the ACK, potentially causing recently established or currently running tasks to be unexpectedly cleared. <img width="1507" height="705" alt="Collector" src="https://github.com/user-attachments/assets/f5470401-9db8-4bda-9f4a-62bf0827a5f6" /> <img width="1509" height="597" alt="Snipaste_2025-12-15_17-42-37" src="https://github.com/user-attachments/assets/6c8017da-cfb5-49cb-be0d-c6fb32a3766c" /> Modification details: 1. Blocked ACK messages from triggering `goOnline()`, ensuring the `Collector` only responds to `ManageServer`'s actively issued (REQUEST) online commands. 2. Add test cases ## Checklist - [x] I have read the [Contributing Guide](https://hertzbeat.apache.org/docs/community/code_style_and_quality_guide) - [ ] I have written the necessary doc or comment. - [x] I have added the necessary unit tests and all cases have passed. ## Add or update API - [ ] I have added the necessary [e2e tests](https://github.com/apache/hertzbeat/tree/master/e2e) and all cases have passed. <img width="1484" height="802" alt="Snipaste_2025-12-15_18-39-26" src="https://github.com/user-attachments/assets/522cefd1-089e-4112-a1af-84f8bf128b5a" /> -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
