-
Notifications
You must be signed in to change notification settings - Fork 84
docs: modify contributing guidelines with AI policy #531
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: main
Are you sure you want to change the base?
Conversation
|
|
||
| Releases are managed by the Apache Iceberg project maintainers. For information about the release process, please refer to the main Iceberg project documentation. | ||
|
|
||
| ## License |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you remove this? It is an unrelated change.
| 1. Finish the intended task. | ||
| 2. Share knowledge between authors and reviewers, as a long-term investment in the project. For this reason, even if someone familiar with the codebase can finish a task quickly, we're still happy to help a new contributor work on it even if it takes longer. | ||
|
|
||
| An AI dump for an issue doesn’t meet these purposes. Maintainers could finish the task faster by using AI directly, and the submitters gain little knowledge if they act only as a pass through AI proxy without understanding. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| An AI dump for an issue doesn’t meet these purposes. Maintainers could finish the task faster by using AI directly, and the submitters gain little knowledge if they act only as a pass through AI proxy without understanding. | |
| An AI dump for an issue doesn't meet these purposes. Maintainers could finish the task faster by using AI directly, and the submitters gain little knowledge if they act only as a pass through AI proxy without understanding. |
Should we stick to ' since it's a plain ASCII?
|
|
||
| Please understand the reviewing capacity is **very limited** for the project, so large PRs which appear to not have the requisite understanding might not get reviewed, and eventually closed or redirected. | ||
|
|
||
| #### Better ways to contribute than an “AI dump” |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| #### Better ways to contribute than an “AI dump” | |
| #### Better ways to contribute than an "AI dump" |
Same question here.
|
Hey @HeartLinked Thanks for bringing this up, and I think it would be good to discuss this and set some clear expectation on AI generated contributions. In case you've missed it, there is an excellent thread going on at the Arrow devlist. I think it would be good to raise a similar discussion on the Iceberg dev-list to form a statement we can share across the different clients (Java, Python, Go, etc). |
Hi @Fokko, we had an off-list discussion that this might be worth discussing further, and we felt it would be worthwhile to raise this topic more broadly within the Iceberg community. I will send an email to dev-list later today. |
kevinjqliu
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
i loveeeeee this
i think this is a great idea that we can use in every iceberg subproject
No description provided.