[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-02-10 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r376953374 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-02-09 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r376882451 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-02-06 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r376236285 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-27 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r371629505 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-27 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r371621340 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369222353 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369222154 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyConditionalSuite.s

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369221877 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyConditionalSuite.s

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369220530 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369219588 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyConditionalSuite.s

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369219245 ## File path: sql/catalyst/src/test/scala/org/apache/spark/sql/catalyst/optimizer/SimplifyConditionalSuite.s

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-21 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r369218678 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks

2020-01-19 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478][SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r368393513 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ###

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-17 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r367834785 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ##

[GitHub] [spark] davidvrba commented on a change in pull request #27231: [SPARK-28478] [SQL] Remove redundant null checks

2020-01-16 Thread GitBox
davidvrba commented on a change in pull request #27231: [SPARK-28478] [SQL] Remove redundant null checks URL: https://github.com/apache/spark/pull/27231#discussion_r367372431 ## File path: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/expressions.scala ##