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

    https://github.com/apache/spark/pull/19295#discussion_r140624028
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/SparkOptimizer.scala ---
    @@ -28,12 +28,18 @@ class SparkOptimizer(
         experimentalMethods: ExperimentalMethods)
       extends Optimizer(catalog) {
     
    -  override def batches: Seq[Batch] = (preOptimizationBatches ++ 
super.batches :+
    +  val experimentalPreOptimizations: Seq[Batch] = Seq(Batch(
    --- End diff --
    
    also define this as `Batch` and you can use `experimentalPreOptimizations 
+: preOptimizationBatches` to concatenate with other batches.


---

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

Reply via email to