alamb commented on code in PR #24510:
URL: https://github.com/apache/datafusion/pull/24510#discussion_r4189585433
##########
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:
Here is the updated version
# AI Policy
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 end-to-end**. Authors should be ready to
justify
and help reviewers understand the design and code during review.
- **Call out unknowns and assumptions**. It's okay to not fully understand
some bits of AI-generated code. Please point these cases out so we can work
together to clear up any concerns.
While "understand the core ideas" is partly subjective, it means more than
being
able to follow the diff textually. We expect the PR author to take an active
role in responding to feedback and crafting the PR to make sure it fits well
into the project as a whole. The worst situation is one where the reviewer
ends
up simply driving the contributor's LLM, for the reasons described in the
next
section.
--
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]