jpopesculian opened a new pull request, #7613: URL: https://github.com/apache/arrow-rs/pull/7613
# Which issue does this PR close? Closes #7612. # What changes are included in this PR? Changes `BoxFuture` to `LocalBoxFuture`. I tried a few implementations like using `async` in the trait or using associated types for the futures, but I find this one to be the easiest to manage, although it unfortunately does not allow for the future to be send if the user so desires. I think generally, using poll and context would make this more flexible, but I didn't want to overcomplicate this PR as generally I think people implement this trait just for use with `AsyncArrowWriter` # Are there any user-facing changes? This is a breaking change as it changes the trait implementation and would be fixed wherever a user implements that trait. It's an easy fix however because `LocalBoxFuture` is always compatible to `BoxFuture` so the user would just have to call `boxed_local` instead of `boxed`. -- 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]
