rickyma commented on PR #1610: URL: https://github.com/apache/incubator-uniffle/pull/1610#issuecomment-2041347438
> I understand your thought, but I have to say that this is an open source project although I hope all changes could not effect the performance and compatiblitiy, it's hard to do. This issue https://github.com/apache/incubator-uniffle/issues/1608 seems to involve quite a few changes. If possible, I hope we can provide more detailed descriptions of the proposed changes in the issue and related PRs, explain why we are making these changes **_in detail_**, and provide a clear background of the problem **_in detail_** for future reference and to facilitate a clearer code review for everyone involved. The current issue and related PR descriptions seem to be a bit too simplistic... especially as time goes by, it becomes inconvenient to understand the background. -- 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]
