UdayanMahalwar commented on issue #49904: URL: https://github.com/apache/arrow/issues/49904#issuecomment-4570415682
Hi @pitrou! I've been diving into Arrow's C++ codebase and would love to take this on as my first PR if it's still available. My plan is to add the ARROW_DEPRECATED macro to the legacy ReadAt and ReadAsync signatures in arrow/io/interfaces.h. Then, I will run a local build and hunt down all the internal usages across the codebase that trigger the deprecation warning, updating them to explicitly pass the allow_short_read boolean. Does that sound like the right approach? If so, I can get started on this right away. -- 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]
