Github user arunmahadevan commented on the issue:
https://github.com/apache/spark/pull/21145
IMO, its better to keep it the current way.
`DataReaderFactory` implies that its something that produces `DataReader`
which it does, whereas `ReadTask` is gives a notion that it does some reading
whereas what it really does is `createDataReader`.
The current naming makes it consistent at the read/write side and I think
the re-naming would add to the already confusing interfaces.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
