2010YOUY01 commented on PR #18237: URL: https://github.com/apache/datafusion/pull/18237#issuecomment-3436468712
> Generally this is great! I think we should get some more feedback but move to merge this quickly and iterate on it if we have to since it's just documentation. > > My personal small suggestion would be to prohibit AI generated comments or responses to review. AI generated summaries in the PR description are probably good, humans myself included are pretty lazy about writing detailed descriptions, AI is great at it and it can end up in the commit message as well which is good for browsing git history. Other than that I expect to be conversing with a human when I ask a question. I don't see any scenario where AI generated comments are helpful, other than for translation / language reasons. I don't feel strongly about this, if there's any opposition let's leave it out. Thanks for the review! +1 for no AI responses to review. However, sometimes I feel AI can't really generate a good PR summary, it's mostly repeating code details, but not to capture the important insights behind it to make the PR more approachable to reviewers. I think it would be an interesting discussion around "How to write a good PR description", like the discussion in https://github.com/apache/datafusion/issues/18095 -- 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]
