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

Reply via email to