UdayanMahalwar commented on code in PR #50061:
URL: https://github.com/apache/arrow/pull/50061#discussion_r3330668650
##########
cpp/src/arrow/io/interfaces.cc:
##########
@@ -151,7 +151,9 @@ RandomAccessFile::RandomAccessFile() : interface_impl_(new
Impl()) {}
Result<int64_t> RandomAccessFile::ReadAt(int64_t position, int64_t nbytes,
bool allow_short_read, void* out) {
+ ARROW_SUPPRESS_DEPRECATION_WARNING
Review Comment:
Since this is the default fallback implementation in the base class, I left
the suppression macros here so it can safely route calls down to the deprecated
2-argument ReadAt method. This ensures backwards compatibility for any external
custom file systems that haven't implemented the new 3-argument signature yet.
--
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]
