[GitHub] spark pull request #21145: [SPARK-24073][SQL]: Rename DataReaderFactory to R...

[dongjoon-hyun <gi...@git.apache.org>]

Github user dongjoon-hyun commented on a diff in the pull request:

    https://github.com/apache/spark/pull/21145#discussion_r184232476
  
    --- Diff: external/kafka-0-10-sql/src/main/scala/org/apache/spark/sql/kafka010/KafkaMicroBatchReader.scala ---
    @@ -299,13 +299,13 @@ private[kafka010] class KafkaMicroBatchReader(
       }
     }
     
    -/** A [[DataReaderFactory]] for reading Kafka data in a micro-batch streaming query. */
    +/** A [[ReadTask]] for reading Kafka data in a micro-batch streaming query. */
     private[kafka010] case class KafkaMicroBatchDataReaderFactory(
    --- End diff --
    
    @rdblue . The re-naming of this PR seems to be partial. Could you replace `KafkaMicroBatchDataReaderFactory` with `KafkaMicroBatchReadTask` together in this PR? I guess there will be more occurrences like this.


---

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