Github user gengliangwang commented on the issue:
    From the PR #20933, we can see that there is a lot of common code between 
`DataReaderFactory[ColumnarBatch]` and `DataReaderFactory[UnsafeRow]`, if we 
use the current method factory pattern.
    This change makes data source implementation easier, and we don't need to 
do runtime type cast.


To unsubscribe, e-mail:
For additional commands, e-mail:

Reply via email to