Github user tdas commented on a diff in the pull request:
https://github.com/apache/spark/pull/19196#discussion_r138756939
--- Diff:
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/IncrementalExecution.scala
---
@@ -117,8 +119,33 @@ class IncrementalExecution(
}
}
- override def preparations: Seq[Rule[SparkPlan]] = state +:
super.preparations
+ override def preparations: Seq[Rule[SparkPlan]] = Seq(
--- End diff --
this is a odd break up of the line. how about
```
override def preparations: Seq[Rule[SparkPlan]] =
Seq(state, EnsureStatefulOpPartitioning) ++ super.preparations
```
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
