LuciferYang commented on PR #42181: URL: https://github.com/apache/spark/pull/42181#issuecomment-1653260851
@panbingkun 1. Please update the pr title due to we upgrade both netty & arrow in this one 2. Please update the PR description and add an explanation for "why we need to upgrade these two dependencies in the one pr?", such as what errors will occur if we only upgrade arrow, and what errors will occur if we only upgrade netty (although I know this is because netty has changed a developer API that arrow is using). It is best to list the pr links or issue links related to netty and arrow -- 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]
