zmuxuny commented on PR #7187: URL: https://github.com/apache/rocketmq-dashboard/pull/7187#issuecomment-6314066269
@lizhimins A short update to the shortlist above, kept in this thread to avoid separate pings: - #6744 has been consolidated into #6745, covering both current topic queries and maintenance pagination after delayed writes. Please use #6745 (`bce99bf`) instead; [current-head CI](https://github.com/apache/rocketmq-dashboard/actions/runs/39415441232) passed. - #6648 (`3a2e5e9`) is also awaiting re-review: the workflow changes and duplicate cleanup were removed after #6676. The remaining two-file diff is environment restoration after extraction failure plus offline regressions; [CI](https://github.com/apache/rocketmq-dashboard/actions/runs/38733428330) passed all five jobs. - #5255 (`af9c528`) retains the requested startup/shutdown coordination, one-second failure backoff, and removal of the redundant started flag. Its [current-head CI](https://github.com/apache/rocketmq-dashboard/actions/runs/38444063676) now passed all five jobs, superseding the older failed-run status in the discussion. - #6654 (`d18e12a`) preserves measured aggregate backlog in health diagnostics when queue observations are absent or partial; [CI](https://github.com/apache/rocketmq-dashboard/actions/runs/38545841541) passed all five jobs. These four are still open, Ready for review and conflict-free when rechecked on October 21. Could you revisit the requested changes on #6648/#5255 and flag any remaining blockers? The other four entries in the earlier shortlist are unchanged. Thanks. -- 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]
