thisisnic commented on PR #50954: URL: https://github.com/apache/arrow/pull/50954#issuecomment-5481902297
> I have a different opinion. So I share my opinion here. > > I think that (non-malicious) reviews from non committers are welcome. (For example, just posting AI generated meaningless reviews are not welcome.) Because: > > * They will reduce maintainers' review costs. > > * These non committers can learn about Apache Arrow and its community from these reviews. > > > Our "Becoming a committer" https://arrow.apache.org/committers/#becoming-committer also includes "reviewing other pull requests": > > > Contributions beyond pull requests, such as reviewing other pull requests, fixing bugs and documentation, triaging issues, answering community questions, improving usability, reducing technical debt, helping with CI, verifying releases, debugging in strange environments, etc. > > How about mentioning any committer when you approve a PR? We may be able to merge PRs faster. > > BTW, I think that Nic's this point is a good suggestion too. > > > happy to point you in the direction of places where it'd be helpful to have extra input if there was a specific area you're most interested in? @kou I'm 100% with you for genuine PR reviews where potential contributors are learning, but this is a very unusual pattern of contributions (i.e. not seen before on this repo) that is leading me to conclude it's AI generated. And there are cases where it's led to ambiguity, where PRs have been "approved", e.g. https://github.com/apache/arrow/pull/50837 I'm massively here for supporting newbies asking questions and giving comments on PRs, but look at this one and I'm just backporting a CHANGELOG entry, so it feels like there's something else going on here beyond someone just learning about the codebase. -- 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]
