paleolimbot commented on PR #965: URL: https://github.com/apache/arrow-nanoarrow/pull/965#issuecomment-6229775774
It's a good point, and (2) does seem safer since it's not an obvious compilation break. I am not sure how that interacts with the existing reader in terms of what needs to be rewritten specifically for the random access reader. As long as this doesn't add a fully duplicate path to the array stream reader (which I think can be done), go for it! > Both would advance the stream position advancing the stream position I think is separate (probably need a seek) -- 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]
