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