HeartSaVioR commented on PR #56057: URL: https://github.com/apache/spark/pull/56057#issuecomment-4527613771
Just 2 cents. AI comment is very very verbose - this PR has only 21 lines of changes and what I got from AI is more than 20 lines of wall of text (most likely more than 50 lines). Can we at least push LLM to be concise and brief, or can we adjust how to post the comment, so that PR author can easily ignore the part of comment? I don't believe PR author needs to read the whole content; major part is for future reviewer. Even worse, if I were to generate the PR description with LLM and don't write it by myself, both of PR description and PR comment would be very verbose and human would have to spend time to read through it. My biggest worry of this is, given the cost of reading through verbose wall of text, PR author will even skip trying to understand PR comments and simply give the PR link to LLM and address review comments, making the whole development loop to be almost human-less. I'm not sure the community is on the same page this is the way to go. -- 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]
