Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/22778#discussion_r227261253
  
    --- Diff: 
sql/catalyst/src/main/scala/org/apache/spark/sql/catalyst/optimizer/Optimizer.scala
 ---
    @@ -171,9 +171,11 @@ abstract class Optimizer(sessionCatalog: 
SessionCatalog)
         // "Extract PythonUDF From JoinCondition".
         Batch("Check Cartesian Products", Once,
           CheckCartesianProducts) :+
    -    Batch("RewriteSubquery", Once,
    +    Batch("Rewrite Subquery", Once,
           RewritePredicateSubquery,
           ColumnPruning,
    +      InferFiltersFromConstraints,
    +      PushDownPredicate,
    --- End diff --
    
    looks good, cc @gatorsmile @maryannxue 


---

---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org
For additional commands, e-mail: reviews-h...@spark.apache.org

Reply via email to