2010YOUY01 commented on code in PR #24510: URL: https://github.com/apache/datafusion/pull/24510#discussion_r4170996663
########## docs/source/contributor-guide/index.md: ########## @@ -129,6 +134,87 @@ Please ensure your PR follows the [testing guide](testing.md). In particular: [Choosing What Kind of Test to Write](testing.md#choosing-what-kind-of-test-to-write). - Run any relevant commands from the [testing quick start](testing.md#testing-quick-start). +## AI-Assisted contributions + +DataFusion has the following policy for AI-assisted PRs: + +- We welcome AI-assisted PRs from anyone. We do not welcome unreviewed "AI dumps" (defined below). +- The PR author should have personally read the entire PR they submit, and **understand the core ideas** behind the implementation **end-to-end**. Authors should be ready to justify and help reviewers understand the design and code during review. Review Comment: I feel “understand the core idea” is a bit vague now, and we could make the expectation more concrete. I also think setting a higher bar for PRs makes it easier to make progress during review. Perhaps ``` “Understand the PR” means more than being able to follow the diff. It means: - Could reproduce the implementation without relying on AI. - Understand how the change fits into the surrounding architecture. - Can judge whether the design adds only necessary complexity and is maintainable long term. ``` -- 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]
