Github user rdblue commented on the issue: https://github.com/apache/spark/pull/21145 @gengliangwang, we can follow up with a rename for the streaming classes that already use this API. But there is no need to do that right now and make this commit larger. I think I've already made it clear that I think `DataReaderFactory` is misleading and we should go with the originally proposed design, `ReadTask`.
--- --------------------------------------------------------------------- To unsubscribe, e-mail: reviews-unsubscr...@spark.apache.org For additional commands, e-mail: reviews-h...@spark.apache.org