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

    https://github.com/apache/spark/pull/22009#discussion_r211808923
  
    --- Diff: 
external/kafka-0-10-sql/src/test/scala/org/apache/spark/sql/kafka010/KafkaContinuousTest.scala
 ---
    @@ -47,7 +47,9 @@ trait KafkaContinuousTest extends KafkaSourceTest {
         eventually(timeout(streamingTimeout)) {
           assert(
             query.lastExecution.logical.collectFirst {
    -          case StreamingDataSourceV2Relation(_, _, _, r: 
KafkaContinuousReader) => r
    +          case r: StreamingDataSourceV2Relation
    +              if r.readSupport.isInstanceOf[KafkaContinuousReadSupport] =>
    +            
r.scanConfigBuilder.build().asInstanceOf[KafkaContinuousScanConfig]
    --- End diff --
    
    good catch!


---

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

Reply via email to