Aias00 opened a new pull request, #2957: URL: https://github.com/apache/rocketmq-dashboard/pull/2957
## Summary - consolidate the native alert collection reliability fixes from #2668 and #2688 - keep a bounded sliding in-flight collection window so saturated executors do not drop later instances - reconcile stale active native alert states only after a successful full collector scope, preserving partial/failure semantics - batch-load latest alert metadata for active state reconciliation to avoid per-state lookups Replaces #2668 Replaces #2688 Closes #2662 Closes #2679 ## Verification - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn -Dtest=CollectorSchedulerTest test` - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn -Dtest=NativeAlertProcessorTest,MybatisPlusAlertStateRepositoryTest test` - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn -Dtest='org.apache.rocketmq.studio.ops.alert.*Test,org.apache.rocketmq.studio.cluster.metrics.*Test' test` mostly passes; it fails only in `AlertSchemaMigrationTest.upgradesExistingAlertTablesWithoutChangingFreshSchemaBehaviorTest` because current base tries to add `rmq_instance_message.result_snapshot` when the legacy fixture lacks `rmq_instance_message`. The same test fails on clean `origin/rocketmq-studio` with the same error. - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn -DskipTests package` - `JAVA_HOME=$(/usr/libexec/java_home -v 21) mvn test` runs 1862 tests with 1 error, the same baseline `AlertSchemaMigrationTest` failure above - `git diff --check origin/rocketmq-studio...HEAD` -- 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]
