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

    https://github.com/apache/spark/pull/21118#discussion_r204150008
  
    --- Diff: 
external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaContinuousReader.scala
 ---
    @@ -53,7 +54,7 @@ class KafkaContinuousReader(
         metadataPath: String,
         initialOffsets: KafkaOffsetRangeLimit,
         failOnDataLoss: Boolean)
    -  extends ContinuousReader with SupportsScanUnsafeRow with Logging {
    --- End diff --
    
    We can, but this is intended to make minimal changes. We can add 
optimizations like this in a follow-up.


---

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

Reply via email to