alamb commented on PR #4676: URL: https://github.com/apache/arrow-rs/pull/4676#issuecomment-1673243811
> > I marked this PR as an api-change as I think it is -- let me know if you disagree > Where is the breaking change you are refering to? I was thinking of https://github.com/apache/arrow-rs/pull/4676/files#r1289997977 but I suppose that since `SyncReader` isn't documented it is unlikely that anyone is using it directly 🤔 > > Finally, I think this PR would be strongly with a test / example of the stated usecase: read metadata once, but create multiple readers for the same file. > https://github.com/apache/arrow-rs/pull/4676/files#diff-850b3a44587149637b8545f66603a2b1252959fd36f7ddc55f37d6b5357816c6R374 As sorry I missed that. -- 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]
