ivasilyev1 commented on PR #59221: URL: https://github.com/apache/spark/pull/59221#issuecomment-6014231346
> My sincere apologize for the back-and-forth on review. I understand your frustration, my bad. > > I think LLM is pushing the code much more complicated and even break the invariant (e.g. source of truth) just to achieve covering more cases to not break, and I feel like we need to step back and draw a line. > > Unfortunately this would delegate some work on mitigation over breakage to users; I hope we could document the issue and the mitigation to guide users for that manual mitigation. It's bad, but I'd accept the trade-off. > > Rationale: After pushing this way, LLM found another blocker/major based on what we have addressed, and I feel like it will continue as long as we just try to self-mitigate with leaving the broken part of it. @HeartSaVioR No -- 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]
