In the future if we use AI in code review, the label also helps us to use a different model which reduces the risk of shared biases and ensures that the reviewer catches flaws the generative model might have overlooked.
On Wed, May 6, 2026 at 2:16 AM Michael Smith <[email protected]> wrote: > I like that framing. > > I've been procrastinating because I'm not sure how the label would change > how we review PRs. It mostly becomes a record of potential copyright > violation if, in the future, that becomes a thing for a particular > tool/model. > > Code review in open-source has to be thorough no matter what, and relies on > trust between people established over time to scale. It's not worth the > review time if the contributor hasn't clearly put in effort to understand > and validate their contribution. > > On Mon, May 4, 2026 at 4:25 PM Quanlong Huang <[email protected]> > wrote: > > > I think "Assisted-by" used in the Linux communities is more > > suitable/accurate. It means "AI is considered a tool (similar to a > compiler > > or static analysis tool) and not an author. The human contributor must > > review, understand, and vouch for every line of code, regardless of > whether > > it was written by them or an AI". > > > > https://docs.kernel.org/process/coding-assistants.html > > https://docs.rockylinux.org/10/guides/contribute/ai-contribution-policy/ > > > > Quanlong > > > > On Thu, Jan 29, 2026 at 10:07 PM Zoltán Borók-Nagy < > [email protected]> > > wrote: > > > > > Thanks, Michael. Yeah, I agree that we should have clear guidelines > and I > > > also don't think we should overuse "Generated-by" if it was only used > for > > > fairly trivial stuff. > > > > > > I also find the scope of "Generated-by" in the commit message too large > > as > > > it doesn't provide any hint which parts were AI generated. > > > > > > I think it would make sense to add "Generated-by" to class/function > > > comments that were mostly generated by AI. > > > > > > - Zoltan > > > > > > On Tue, Jan 27, 2026 at 9:59 PM Michael Smith < > > [email protected]> > > > wrote: > > > > > > > These guidelines seem to leave some of the specifics up to individual > > > > projects. I think we should agree on some clearer guidelines for > > > > contributors, and post it on our wiki > > > > < > > > > > > https://cwiki.apache.org/confluence/display/IMPALA/Contributing+to+Impala > > > > >. > > > > > > > > I do have GitHub Copilot enabled, and my rule-of-thumb is if I do > more > > > than > > > > accept small auto-complete suggestions (not much different to what > > > > pre-generative tooling produced) that I can immediately understand, I > > add > > > > "Generated-by: GitHub Copilot (<model>)" to the commit message, like > in > > > > apache/impala#ac1c11dd > > > > < > > > > > > > > > > https://github.com/apache/impala/commit/ac1c11dd8256e8a81e138f43663de06610441d41 > > > > > > > > > or apache/impala#c0b3580 > > > > < > > > > > > > > > > https://github.com/apache/impala/commit/c0b35807543f96cb382240717ab552d17f6f78a6 > > > > > > > > > . > > > > > > > > On Mon, Dec 15, 2025 at 4:39 AM Zoltán Borók-Nagy < > > [email protected] > > > > > > > > wrote: > > > > > > > > > Dear Contributors, > > > > > > > > > > With AI tools getting more popular these days, please take some > time > > to > > > > > read the ASF Generative Tooling Guidance: > > > > > https://www.apache.org/legal/generative-tooling.html > > > > > > > > > > Kind regards, > > > > > Zoltan Borok-Nagy > > > > > > > > > > > > > > >
