LuciferYang commented on PR #42236: URL: https://github.com/apache/spark/pull/42236#issuecomment-1667799611
@srowen No, this is not the impact brought by this pull request, it's just the current state of the `spark-protobuf` module. The purpose of this pull request is to make both sbt and maven pass the test under the current state of the `spark-protobuf` module. The current state of the `spark-protobuf` module mentioned here is similar to what you understood, with specific details similar to the comments in the following function: https://github.com/apache/spark/blob/433fb2af8a3ca239958cb7b006e2924ecfac0d56/connector/protobuf/src/main/scala/org/apache/spark/sql/protobuf/functions.scala#L118-L136 This is unrelated to Spark Connect itself, Spark Connect just proxies this functions -- 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]
