MaxGekk commented on a change in pull request #32176:
URL: https://github.com/apache/spark/pull/32176#discussion_r614303863



##########
File path: 
sql/core/src/test/scala/org/apache/spark/sql/streaming/StreamingJoinSuite.scala
##########
@@ -47,6 +48,10 @@ abstract class StreamingJoinSuite
     StateStore.stop()
   }
 
+  override protected def sparkConf = super.sparkConf
+    // TODO(SPARK-35095): Use ANSI intervals in streaming join tests

Review comment:
       For some reasons, `CalendarInterval` is used in the streaming code. For 
instance at
   
https://github.com/apache/spark/blob/0945baf90660a101ae0f86a39d4c91ca74ae5ee3/sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/EventTimeWatermarkExec.scala#L89-L91
   
   Need to revise the code. The streaming tests fails on new ANSI intervals at 
the moment.




-- 
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.

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