LuciferYang commented on PR #42236: URL: https://github.com/apache/spark/pull/42236#issuecomment-1687513722
@HyukjinKwon Hmm... I think the fix should be fine. However, as Spark 3.5 no longer needs to support protobuf 2.5, I hope to make some changes to the `Experimental` Apis involved in this pr to make it no longer intrude into user code. The related viewpoints include： https://github.com/apache/spark/pull/42236#issuecomment-1664828251, https://github.com/apache/spark/pull/42236#issuecomment-1665167038, https://github.com/apache/spark/pull/42236#issuecomment-1668006357 Of course, we could also choose to fix this issue for Spark 3.5 using this pr first and have more discussions or changes for this in Spark 4.0. -- 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]
