tustvold commented on code in PR #5135:
URL: https://github.com/apache/arrow-rs/pull/5135#discussion_r1409105685
##########
parquet/src/arrow/async_reader/mod.rs:
##########
@@ -385,13 +385,28 @@ impl<T: AsyncFileReader + Send + 'static>
ParquetRecordBatchStreamBuilder<T> {
offset: self.offset,
};
+ // Ensure schema of ParquetRecordBatchStream respects projection, and
does
+ // not store metadata (same as for ParquetRecordBatchReader and
emitted RecordBatches)
+ let projected_fields = match reader.fields.as_deref().map(|pf|
&pf.arrow_type) {
Review Comment:
This isn't quite correct in the case of a projection of a nested schema, its
possible this isn't actually determined until the ArrayReader is constructed...
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
