zuston commented on PR #1610: URL: https://github.com/apache/incubator-uniffle/pull/1610#issuecomment-2041351661
> > 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 #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 beta feature is still on-going(not a producation feature) and if you want to dig more, you can find more from the issue link. But if you want to know why this happen, I think the test case will show somthing you want to know. And for your reviewing if you want, feel free to discuss more about the design and fix / improvement. -- 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]
