Kwafoor commented on pull request #34862:
URL: https://github.com/apache/spark/pull/34862#issuecomment-992072302


   > @Kwafoor the default behavior exists for many years. It is hard to change 
it now. That's also why we introducing the ANSI SQL mode as an alternative 
option. If users want to detect integer overflow, they can enable the 
`spark.sql.ansi.enabled`.
   
   I agree with you, but I think SparkSQL should remind user your sql is wrong 
rather return null.
   User may not check their results, it will lead a hug trouble.
   ANSI with default true is also not a good idea, it is too strict with cast.


-- 
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]

Reply via email to