[GitHub] [spark] cloud-fan commented on a change in pull request #28962: [SPARK-32136][SQL] NormalizeFloatingNumbers should work on null struct

2020-07-06 Thread GitBox
cloud-fan commented on a change in pull request #28962: URL: https://github.com/apache/spark/pull/28962#discussion_r450162453 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/NormalizeFloatingNumbers.scala ## @@ -123,7 +123,8 @@ object Normaliz

[GitHub] [spark] cloud-fan commented on a change in pull request #28962: [SPARK-32136][SQL] NormalizeFloatingNumbers should work on null struct

2020-07-01 Thread GitBox
cloud-fan commented on a change in pull request #28962: URL: https://github.com/apache/spark/pull/28962#discussion_r448526240 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/NormalizeFloatingNumbers.scala ## @@ -123,7 +123,8 @@ object Normaliz

[GitHub] [spark] cloud-fan commented on a change in pull request #28962: [SPARK-32136][SQL] NormalizeFloatingNumbers should work on null struct

2020-07-01 Thread GitBox
cloud-fan commented on a change in pull request #28962: URL: https://github.com/apache/spark/pull/28962#discussion_r44851 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/NormalizeFloatingNumbers.scala ## @@ -123,7 +123,8 @@ object Normaliz

[GitHub] [spark] cloud-fan commented on a change in pull request #28962: [SPARK-32136][SQL] NormalizeFloatingNumbers should work on null struct

2020-07-01 Thread GitBox
cloud-fan commented on a change in pull request #28962: URL: https://github.com/apache/spark/pull/28962#discussion_r448155485 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/NormalizeFloatingNumbers.scala ## @@ -123,7 +123,8 @@ object Normaliz