comphead commented on code in PR #10312: URL: https://github.com/apache/datafusion/pull/10312#discussion_r1585066972
########## docs/source/contributor-guide/index.md: ########## @@ -68,10 +68,12 @@ ideas with the community to get feedback on implementation. We welcome pull requests (PRs) from anyone from the community. -DataFusion is a very active fast-moving project and we try to review and merge PRs quickly to keep the review backlog down and the pace up. After review and approval, one of the [many people with commit access](https://arrow.apache.org/committers/) will merge your PR. +DataFusion is a very active fast-moving project and we try to review and merge PRs quickly to keep the review backlog down and the pace up. After review and approval, one of the [many people with commit access] will merge your PR. Review bandwidth is currently our most limited resource, and we highly encourage reviews by the broader community. If you are waiting for your PR to be reviewed, consider helping review other PRs that are waiting. Such review both helps the reviewer to learn the codebase and become more expert, as well as helps identify issues in the PR (such as lack of test coverage), that can be addressed and make future reviews faster and more efficient. +[many people with commit access]: https://people.apache.org/phonebook.html?unix=datafusion Review Comment: ```suggestion [commiters]: https://people.apache.org/phonebook.html?unix=datafusion ``` ########## docs/source/contributor-guide/index.md: ########## @@ -68,10 +68,12 @@ ideas with the community to get feedback on implementation. We welcome pull requests (PRs) from anyone from the community. -DataFusion is a very active fast-moving project and we try to review and merge PRs quickly to keep the review backlog down and the pace up. After review and approval, one of the [many people with commit access](https://arrow.apache.org/committers/) will merge your PR. +DataFusion is a very active fast-moving project and we try to review and merge PRs quickly to keep the review backlog down and the pace up. After review and approval, one of the [many people with commit access] will merge your PR. Review bandwidth is currently our most limited resource, and we highly encourage reviews by the broader community. If you are waiting for your PR to be reviewed, consider helping review other PRs that are waiting. Such review both helps the reviewer to learn the codebase and become more expert, as well as helps identify issues in the PR (such as lack of test coverage), that can be addressed and make future reviews faster and more efficient. +[many people with commit access]: https://people.apache.org/phonebook.html?unix=datafusion Review Comment: ```suggestion [committers]: https://people.apache.org/phonebook.html?unix=datafusion ``` -- 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: github-unsubscr...@datafusion.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: github-unsubscr...@datafusion.apache.org For additional commands, e-mail: github-h...@datafusion.apache.org
