stevqin commented on PR #18648: URL: https://github.com/apache/dolphinscheduler/pull/18648#issuecomment-5728486105
Addressed the requested live-update performance changes in `734ccd9`. - Indexed the Gantt response by task name in one pass before building rows. Precision matching now searches only the indexed candidates while retaining the existing duplicate-name and status checks. - Cached the localized `tasksState(t)` mapping instead of recreating it for every rendered row. - Added a regression test with 100 tasks. Before the change, model construction read 5,051 Gantt task entries; the indexed implementation performs a linear scan. Verification: - `node --test tests/gantt*.test.cjs`: 10/10 passed - `vue-tsc --noEmit`: passed - Prettier check for the changed files: passed - `npm run build:prod`: passed (existing bundle-size and `eval` warnings remain) Please re-review when convenient. -- 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]
