alamb commented on issue #6099: URL: https://github.com/apache/arrow-rs/issues/6099#issuecomment-2242823970
Thanks @V0ldek I personally suggest we make both changes (as two separate PRs) Having a `flush` makes a lot of sense and would make the API more flexible and is backwards compatible In terms of modifying the internal buffering, I think we should consider removing the internal buffering entirely (and replace it with some documentation / examples that shows how to do it) Removing the internal buffering is an API change as you say, but I think it is more inline with rust `std` APIs where the user retains as much control over the buffering (e.g. by explicitly using `BufWriter`) as possible -- 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]
