GitHub user leborchuk added a comment to the discussion: Introducing the [perfmon] Extension for Cloudberry Database Monitoring
Hi @fanfuxiaoran, very thanks for your work! I have a thought on how to properly review and merge such complex functionality. Since we have agreed not to merge it into 2.0.0, what about creating a separate branch? In this branch, we can merge the initial set of changes, check them, discuss them, create additional PRs, and then merge the final PR into the main branch. I was reviewing https://github.com/apache/cloudberry/pull/1085, but realized that if we started discussing the implementation details, the review process would be delayed. Plus, we might postpone some ideas and come back to them later. However, they might get lost in the general discussion flow. It would be better to write them down and return to them at a later date. GitHub link: https://github.com/apache/cloudberry/discussions/1087#discussioncomment-13142837 ---- This is an automatically sent email for dev@cloudberry.apache.org. To unsubscribe, please send an email to: dev-unsubscr...@cloudberry.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@cloudberry.apache.org For additional commands, e-mail: dev-h...@cloudberry.apache.org
