haohuaijin commented on PR #10702: URL: https://github.com/apache/arrow-rs/pull/10702#issuecomment-5302965985
> In my mind the trick will be how to mesh the existing apis (with_row_groups and with_row_selection) and make sure we have a coherent documentation of what will happen if the user calls each one This PR adds `ParquetPushDecoderBuilder::with_row_group_selections` so callers can supply row-group-local RowSelections directly. The new configuration is mutually exclusive with `with_row_groups` / `with_row_selection`: an internal state machine catches any conflicting combination in any call order and reports it as an error from `build()`, the legacy APIs keep their exact semantics, and each setter's docs spell out how it interacts with the others. cc @alamb @zhuqi-lucas -- 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]
