You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/02/19 21:11:00 UTC

[GitHub] [arrow-rs] gsserge commented on pull request #1341: Make bounds configurable in csv ReaderBuilder

gsserge commented on pull request #1341:
URL: https://github.com/apache/arrow-rs/pull/1341#issuecomment-1046103503


   It might be better from the API perspective for the `with_bounds()` method to have explicit `start` and `end` parameters instead of wrapping a tuple in `Some`: `.with_bounds(0, 2)` instead of `.with_bounds(Some((0, 2)))`.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org