wgtmac commented on code in PR #18752: URL: https://github.com/apache/iceberg/pull/18752#discussion_r4499649114
########## site/docs/community.md: ########## @@ -143,7 +143,9 @@ Many contributors have questions about how to become a committer. This section o ### What are the responsibilities of a committer? -In the Iceberg project, committers are community members that can review and commit changes to Iceberg repositories. Reviewing is the primary responsibility of committers. +Committers are project reviewers and are able to commit code in Iceberg repositories. Reviewing is the primary responsibility of committers. + +Although ASF write access applies across all project repositories, committers are trusted to merge PRs or commit only in areas and Iceberg implementations where they are knowledgeable. When contributing to or reviewing changes outside their area of expertise or in another implementation, committers should work with maintainers of that codebase and get a review from someone familiar with the area before merging. Review Comment: Agreed. It is vague to use `maintainers` here. -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
