gaogaotiantian commented on PR #56174: URL: https://github.com/apache/spark/pull/56174#issuecomment-5631345457
Yes, I was wondering if raising a pyspark exception is better than what we have now. Matching the scala behavior could be a motivation, but is the scenario the same? I did not check the scala side code. We don't raise any pyspark exception in `pyspark.sql.connect.protobuf` and that is a kind of "consistency" - not saying that it's the correct one. That's the part I need some convincing. -- 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]
