[GitHub] spark pull request #19947: [SPARK-22759] [SQL] Filters can be combined iff b...

2017-12-12 Thread asfgit
Github user asfgit closed the pull request at: https://github.com/apache/spark/pull/19947 --- - To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org

[GitHub] spark pull request #19947: [SPARK-22759] [SQL] Filters can be combined iff b...

2017-12-11 Thread gengliangwang
Github user gengliangwang commented on a diff in the pull request: https://github.com/apache/spark/pull/19947#discussion_r156250376 --- Diff: sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala --- @@ -695,7 +695,9 @@ object CombineUnions extends

[GitHub] spark pull request #19947: [SPARK-22759] [SQL] Filters can be combined iff b...

2017-12-11 Thread gatorsmile
GitHub user gatorsmile opened a pull request: https://github.com/apache/spark/pull/19947 [SPARK-22759] [SQL] Filters can be combined iff both are deterministic ## What changes were proposed in this pull request? The query execution/optimization does not guarantee the expressions