Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21599#discussion_r199632638
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -128,17 +128,31 @@ abstract class BinaryArithmetic
Github user mgaido91 commented on a diff in the pull request:
https://github.com/apache/spark/pull/21599#discussion_r197246916
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -128,17 +128,31 @@ abstract class BinaryArithmetic
Github user cloud-fan commented on a diff in the pull request:
https://github.com/apache/spark/pull/21599#discussion_r197235683
--- Diff:
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala
---
@@ -128,17 +128,31 @@ abstract class BinaryArithmeti
GitHub user mgaido91 opened a pull request:
https://github.com/apache/spark/pull/21599
[SPARK-24598][SQL] Overflow on arithmetic operations returns incorrect
result
## What changes were proposed in this pull request?
When an overflow occurs performing an arithmetic operatio
