[GitHub] [spark] sandeep-katta commented on a change in pull request #28600: [SPARK-31761][SQL] cast integer to Long to avoid IntegerOverflow for IntegralDivide operator

2020-05-22 Thread GitBox
sandeep-katta commented on a change in pull request #28600: URL: https://github.com/apache/spark/pull/28600#discussion_r429519637 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/analysis/TypeCoercion.scala ## @@ -684,6 +685,24 @@ object TypeCoercion {

[GitHub] [spark] sandeep-katta commented on a change in pull request #28600: [SPARK-31761][SQL] cast integer to Long to avoid IntegerOverflow for IntegralDivide operator

2020-05-22 Thread GitBox
sandeep-katta commented on a change in pull request #28600: URL: https://github.com/apache/spark/pull/28600#discussion_r429088285 ## File path: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala ## @@ -3495,6 +3495,13 @@ class SQLQuerySuite extends QueryTest with

[GitHub] [spark] sandeep-katta commented on a change in pull request #28600: [SPARK-31761][SQL] cast integer to Long to avoid IntegerOverflow for IntegralDivide operator

2020-05-22 Thread GitBox
sandeep-katta commented on a change in pull request #28600: URL: https://github.com/apache/spark/pull/28600#discussion_r429083860 ## File path: sql/core/src/test/scala/org/apache/spark/sql/SQLQuerySuite.scala ## @@ -3495,6 +3495,13 @@ class SQLQuerySuite extends QueryTest with

[GitHub] [spark] sandeep-katta commented on a change in pull request #28600: [SPARK-31761][SQL] cast integer to Long to avoid IntegerOverflow for IntegralDivide operator

2020-05-21 Thread GitBox
sandeep-katta commented on a change in pull request #28600: URL: https://github.com/apache/spark/pull/28600#discussion_r429034719 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala ## @@ -337,11 +340,19 @@ trait DivModLike exte

[GitHub] [spark] sandeep-katta commented on a change in pull request #28600: [SPARK-31761][SQL] cast integer to Long to avoid IntegerOverflow for IntegralDivide operator

2020-05-21 Thread GitBox
sandeep-katta commented on a change in pull request #28600: URL: https://github.com/apache/spark/pull/28600#discussion_r429034382 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala ## @@ -337,11 +340,19 @@ trait DivModLike exte

[GitHub] [spark] sandeep-katta commented on a change in pull request #28600: [SPARK-31761][SQL] cast integer to Long to avoid IntegerOverflow for IntegralDivide operator

2020-05-21 Thread GitBox
sandeep-katta commented on a change in pull request #28600: URL: https://github.com/apache/spark/pull/28600#discussion_r429033795 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/expressions/arithmetic.scala ## @@ -337,11 +340,19 @@ trait DivModLike exte