As part of the recent reader work we introduced
but in datafusion we have
These two trait are almost identical (BatchIterator implements Send + Sync whereas RecordBatchReader does not). I propose we replace RecordBatchReader with BatchIterator (i.e. move it to arrow as it's generally useful outside of datafusion) and update parquet and data fusion accordingly.