Hi all,
I hope this message finds you well. I am writing to propose an enhancement to our current main branch protection rules in the IoTDB repository. Specifically, I suggest modifying the branch protection settings to require at least two approving reviews from contributors before code can be merged into the main branch. Why This Change? 1. Improved Code Quality: Multiple reviews help catch issues—such as logical errors, edge cases, or performance regressions—that might be overlooked in a single review. This aligns with Apache Software Foundation’s emphasis on collaborative quality assurance. 1. Knowledge Sharing: Encouraging more reviewers fosters broader understanding of code changes across the team, reducing bus-factor risks. 1. Community Inclusivity: Involving more contributors in reviews promotes mentorship and aligns with Apache’s "Community Over Code" principle. Current Gap While the main branch may already have some protection, the absence of a mandatory multi-reviewer step could allow potential issues to slip through, especially for complex changes. Proposed Implementation * Update branch protection rules (e.g., in GitHub) to require ≥2 approvals. * This practice is common in ASF projects has proven to reduce integration risks. I believe it would strengthen IoTDB’s reliability as it evolves toward cloud-native and edge scenarios . I welcome feedback and would be happy to help implement the change. Best regards, Xuan Wang
