kou commented on PR #50954: URL: https://github.com/apache/arrow/pull/50954#issuecomment-5416865334
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? -- 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]
