alamb commented on issue #8643:
URL: https://github.com/apache/arrow-rs/issues/8643#issuecomment-3487600095
> Not a show stopper, but to avoid breaking changes we'll need to be
careful. I'd imagine many of the options on the ArrowReaderBuilder (projection,
row_groups, etc) would need to move to the metadata options, but be left in the
current API with deprecation notices and a warning that setting such options
with both the ArrowReaderBuilder and MetadataOptions may lead to UB.
I am not 100% sure we should consolidate the options
Specifically, I think the reader builders should be constructible with
pre-existing metadata as well (e.g. when it was cached), so there needs to be
some way to represent the "read only X row group" even when we already have the
ParquetMetadata I think
--
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]
