Github user wzhfy commented on a diff in the pull request:

    https://github.com/apache/spark/pull/19295#discussion_r140623965
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/ExperimentalMethods.scala ---
    @@ -44,11 +44,14 @@ class ExperimentalMethods private[sql]() {
        */
       @volatile var extraStrategies: Seq[Strategy] = Nil
     
    +  @volatile var extraPreOptimizations: Seq[Rule[LogicalPlan]] = Nil
    +
       @volatile var extraOptimizations: Seq[Rule[LogicalPlan]] = Nil
    --- End diff --
    
    how about rename this `extraPostOptimizations`?


---

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

Reply via email to