pitrou opened a new issue, #49904: URL: https://github.com/apache/arrow/issues/49904
### Describe the enhancement requested In PR https://github.com/apache/arrow/pull/49897 we introduced overloads of `RandomAccessFile::ReadAt` and `RandomAccessFile::ReadAsync` with an additional `bool allow_short_read` argument. This ensures that the caller is conscious about the potential return value of these methods when the file is not large enough to satisfy the entire read. The legacy `RandomAccessFile::ReadAt` and `RandomAccessFile::ReadAsync` methods without the new argument are slightly dangerous as the caller then has to check the length of the returned result. We should deprecate them and schedule them for removal in a bunch of versions (perhaps in 4 versions to give some time for callers to migrate their API usage?). ### Component(s) C++ -- 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]
