-
Notifications
You must be signed in to change notification settings - Fork 3.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ARROW-16226: [C++] Add better coverage for filesystem tell. #14064
Conversation
Looks like some of the new assertions are firing for |
This sounds like a bug that deserves fixing. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @benibus ! This looks good, just two minor suggestions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Could you please rebase on git master to fix the macOS CI failures?
af17b3d
to
5ef3e28
Compare
I see CI tests have passed on your fork, so I'm gonna merge now. |
Benchmark runs are scheduled for baseline = 4cb7b50 and contender = 44ae852. 44ae852 is a master commit associated with this PR. Results will be available as each benchmark for each run completes. |
…4064) Based on [ARROW-16226](https://issues.apache.org/jira/browse/ARROW-16226). Adds coverage to GenericFileSystemTest::TestOpenInput(Stream|File) for validating Tell() and reads after seeking. Authored-by: benibus <bpharks@gmx.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
…4064) Based on [ARROW-16226](https://issues.apache.org/jira/browse/ARROW-16226). Adds coverage to GenericFileSystemTest::TestOpenInput(Stream|File) for validating Tell() and reads after seeking. Authored-by: benibus <bpharks@gmx.com> Signed-off-by: Antoine Pitrou <antoine@python.org>
Based on ARROW-16226.
Adds coverage to GenericFileSystemTest::TestOpenInput(Stream|File) for validating Tell() and reads after seeking.