HeartSaVioR commented on code in PR #38405:
URL: https://github.com/apache/spark/pull/38405#discussion_r1007386332


##########
sql/catalyst/src/main/scala/org/apache/spark/sql/internal/SQLConf.scala:
##########
@@ -1941,6 +1941,22 @@ object SQLConf {
       .booleanConf
       .createWithDefault(true)
 
+  val STATEFUL_OPERATOR_ALLOW_MULTIPLE =
+    buildConf("spark.sql.streaming.statefulOperator.allowMultiple")
+      .internal()
+      .doc("When true, multiple stateful operators are allowed to be present 
in a streaming " +
+        "pipeline. The support for multiple stateful operators introduces a 
minor (semantically " +
+        "correct) change in respect to late record filtering - late records 
are detected and " +
+        "filtered in respect to the watermark from the previous microbatch 
instead of the " +
+        "current one. This is a behavior change for Spark streaming pipelines 
and we allow " +

Review Comment:
   Yes, we will have a bunch of errors in test suites when we disable no-data 
batch. Most test cases are assuming that no-data batch always happens.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to