SEZ9 commented on issue #11666: URL: https://github.com/apache/seatunnel/issues/11666#issuecomment-5225135242
Thanks for the well-researched proposal. Since the windowMs-parameterized endpoints and the drawer table already exist, this looks like a UI-focused change. Could you propose which charting approach to use (e.g., ECharts, which is common in Apache UIs) and the default refresh interval/window size so we can review before implementation starts? The proposed scope text appears cut off after 'so the chart'. Could you complete the scope section, especially regarding: whether pinned selections should persist across page navigation/refresh, any limit on the number of pinned series, and whether the chart replaces or coexists with the current table view? Are you interested in contributing a PR? If so, please note this may relate to #11351/#11352 (inline metrics on the graph) - coordinating on shared metric-fetching logic in seatunnel-engine-ui would avoid duplicated polling of the realtime endpoints. <!-- streview-comment:78 --> -- 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]
