Github user xuanyuanking commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21560#discussion_r195420149
  
    --- Diff: 
sql/core/src/main/scala/org/apache/spark/sql/execution/datasources/v2/DataSourceV2Strategy.scala
 ---
    @@ -36,6 +37,17 @@ object DataSourceV2Strategy extends Strategy {
         case WriteToContinuousDataSource(writer, query) =>
           WriteToContinuousDataSourceExec(writer, planLater(query)) :: Nil
     
    +    case Repartition(1, false, child) =>
    +      val isContinuous = child.collectFirst {
    +        case StreamingDataSourceV2Relation(_, _, _, r: ContinuousReader) 
=> r
    +      }.isDefined
    --- End diff --
    
    The judgement of whether the plan is continuous or not can be a sperated 
method and other place can use it?


---

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

Reply via email to