zmuxuny commented on PR #4457: URL: https://github.com/apache/rocketmq-dashboard/pull/4457#issuecomment-5771213294
Addressed the blocking lifecycle items in `9380d072` on `rocketmq-studio@4c697f0`. The explicit policy is now disable -> purge, enable -> preserve, semantic edit -> purge, and presentation-only edit of an enabled rule -> preserve. Both toggle overloads, both bulk paths and both edit entrypoints are covered. Delete still purges; disabling does not emit a synthetic recovery. A new stateful runtime regression first observes FIRING plus reminder timestamps and then requires the disabled rule's runtime view to be empty. All six disable variants failed against the previous implementation (20 lifecycle cases, 6 failures); the revised lifecycle suite is 20/20. The combined lifecycle/service/fingerprint/native-processor suite is 130/130, Checkstyle 0 and package succeeds. The PR body now explicitly explains the enable-direction contract change and the retained disable/delete assertions. The optional duplicate lookup optimization is left separate to keep this revision focused. No full external-datasource integration run is claimed. -- 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]
