Kwafoor commented on pull request #34862: URL: https://github.com/apache/spark/pull/34862#issuecomment-992069687
> @Kwafoor, Before we go further, I think it should be best to double check how ANSI defines the behaviour on the overflow of strings <> other types. > @Kwafoor Please enable `spark.sql.ansi.enabled` if you prefer thrown exception. With spark.sql.ansi.enabled enable it is right, but ANSI is too strict with cast, user often didn't enable. Without thrown Exception, when overflow happened SparkSQL return wrong results(null), but user didn't get any remind, I think it is a problem. Without thrown exception it is also too hard to find where the wrong sql is and fix it. -- 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]
