Github user cloud-fan commented on a diff in the pull request:

    https://github.com/apache/spark/pull/20598#discussion_r168064805
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/streaming/StreamingRelation.scala
 ---
    @@ -62,7 +64,7 @@ case class StreamingRelation(dataSource: DataSource, 
sourceName: String, output:
     case class StreamingExecutionRelation(
    --- End diff --
    
    not very familiar with the streaming side, but IIRC, some of these plans 
are temporary and will be replaced before entering analyzer, and these plans 
don't need to extend MultiInstanceRelation.


---

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

Reply via email to