tju-yxq opened a new pull request, #2697: URL: https://github.com/apache/rocketmq-dashboard/pull/2697
## What changed A single native alert evaluation could abort the rest of a collection batch. `NativeAlertProcessor` used one transaction for all matching samples and rules, so an aggregation, state, alert, or notification failure stopped subsequent evaluations and could roll back earlier work. This change keeps rule loading at the batch/domain level, but moves each rule/sample evaluation into `NativeAlertEvaluationService` with its own `REQUIRES_NEW` transaction. A failed evaluation is logged with bounded context and the processor continues with the remaining rules and samples. The existing aggregation, state-transition, suppression, lifecycle-event, and outbox behavior is preserved. Fixes #2690 ## Verification - `mvn -q '-Dtest=NativeAlertProcessorTest,CollectorSchedulerTest,StudioApplicationTest' test` - `mvn -DskipTests=false test` (1792 tests, 0 failures, 0 errors, 0 skipped) - `git diff --check` The tests are deterministic unit/context tests; no live broker or cloud service was used. -- 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]
