alamb commented on PR #10354: URL: https://github.com/apache/arrow-rs/pull/10354#issuecomment-5113613895
> I personally don't think it is all that hard to justify that we provide a batteries included option using the IO abstraction maintained as part of the arrow project, and have APIs to allow plugging in other IO abstractions should users wish to use them. That being said I was mainly responding to the linked ticket, as if the issue is upgrade hazards, I fail to see that this PR actually is an improvement in that regard - if anything it will make it worse as each downstream project is forced to adopt their own variant of it. Given this feedback the more I am liking the approach of moving the object_store code to a new crate -- this would mean we still provide a first party integration but it would be an opt-in dependency 🤔 -- 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]
