WweiL commented on code in PR #41026:
URL: https://github.com/apache/spark/pull/41026#discussion_r1194154060


##########
sql/core/src/main/scala/org/apache/spark/sql/streaming/DataStreamWriter.scala:
##########
@@ -455,6 +465,16 @@ final class DataStreamWriter[T] private[sql](ds: 
Dataset[T]) {
     this
   }
 
+  private[sql] def foreachConnect(writer: PythonForeachWriter): 
DataStreamWriter[T] = {

Review Comment:
   Yes actually this change is due to the change I added in scala foreach: 
https://github.com/apache/spark/pull/41129/files#r1194152295
   
   The writer will be a generic ForeachWriter[Any]. Here I put 
PythonForeachWriter because at this stage it doesn't need to be generic



-- 
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: reviews-unsubscr...@spark.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


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

Reply via email to