GitHub user zuozhiw added a comment to the discussion: Approach: landing Modern Observability (#4070) as a stack of small, self-contained PRs
Hi @Ma77Ball i like the approach of having two separate tracks, it looks good to me. I would say even if the diff is big, don't try to make too many Ps, eg 18 stacked PRs makes logic fragmented and might be even harder to review. I would encourage to make each pr to be more focused on a sub part but to be more complete. Current proposal sounds too fragmented to me. Apart from that, I'm not following the context for the angular dashboard. Are we building our own UI for displaying observability stuff? I thought we are replying on third party vendors like grafana. Please let me know what do those changes include. Thx GitHub link: https://github.com/apache/texera/discussions/5355#discussioncomment-17174822 ---- This is an automatically sent email for [email protected]. To unsubscribe, please send an email to: [email protected]
